Skip to content

Conversation

@codixor
Copy link

@codixor codixor commented Nov 24, 2025

Fix Error handling of get_devices_gql()

Summary by CodeRabbit

  • Chores
    • Updated alexapy library dependency to version 1.29.10

✏️ Tip: You can customize this high-level summary in your review settings.

Fix Error handling of get_devices_gql()
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 24, 2025

Walkthrough

The alexa_media component's manifest.json updates the alexapy library dependency from version 1.29.9 to 1.29.10. Dependencies on packaging and wrapt remain unchanged.

Changes

Cohort / File(s) Summary
Dependency version bump
custom_components/alexa_media/manifest.json
Updated alexapy requirement from 1.29.9 to 1.29.10

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Poem

🔧 A version bump, precise and clean,
From 1.29.9 to ten,
Alexapy refreshed,
Requirements blessed.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately reflects the main change: bumping the alexapy dependency from 1.29.9 to 1.29.10 to fix error handling in get_devices_gql().
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 027ec84 and 16092ca.

📒 Files selected for processing (1)
  • custom_components/alexa_media/manifest.json (1 hunks)

@codixor codixor changed the title Update alexapy requirement to version 1.29.10 fix:Update alexapy requirement to version 1.29.10 Nov 24, 2025
@codixor codixor changed the title fix:Update alexapy requirement to version 1.29.10 fix: Update alexapy requirement to version 1.29.10 Nov 24, 2025
@danielbrunt57
Copy link
Collaborator

danielbrunt57 commented Nov 26, 2025

@alandtse,
There is a new important AlexaPy MR !430- fix: /api/notifications is returning {"message":"Rate exceeded"} response on every post.

Perhaps this can be delayed and instead bump alexapy to 1.29.11?

@alandtse alandtse changed the title fix: Update alexapy requirement to version 1.29.10 fix: bump alexapy to 1.29.10 Nov 30, 2025
Copy link
Owner

@alandtse alandtse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. This is an incomplete bump. Please handle all instances of alexapy including the lock. Look at past bumps.

@danielbrunt57
Copy link
Collaborator

Hi @codixor,
Maybe this PR should be updated to alexapy Release 1.29.11?
Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants