Skip to content

Conversation

@jamesonuk
Copy link
Contributor

Proposed change

Bring documentation up to date and provide additional examples.

Type of change

  • Spelling, grammar or other readability improvements (current branch).
  • Adjusted missing or incorrect information in the current documentation (current branch).
  • Added documentation for a new integration I'm adding to Home Assistant (next branch).
  • Added documentation for a new feature I'm adding to Home Assistant (next branch).
  • Removed stale or deprecated documentation.

Additional information

  • Link to parent pull request in the codebase:
  • Link to parent pull request in the Brands repository:
  • This PR fixes or closes issue: fixes #

Checklist

  • This PR uses the correct branch, based on one of the following:
    • I made a change to the existing documentation and used the current branch.
    • I made a change that is related to an upcoming version of Home Assistant and used the next branch.
  • The documentation follows the Home Assistant documentation standards.

@home-assistant home-assistant bot added the current This PR goes into the current branch label Nov 5, 2025
@netlify
Copy link

netlify bot commented Nov 5, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit e13fbca
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/690e34d61354600007d2f7b4
😎 Deploy Preview https://deploy-preview-41622--home-assistant-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

chemelli74
chemelli74 previously approved these changes Nov 6, 2025
Copy link
Contributor

@chemelli74 chemelli74 left a comment

Choose a reason for hiding this comment

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

Thx @jamesonuk !

@frenck frenck marked this pull request as ready for review November 7, 2025 11:45
Copilot AI review requested due to automatic review settings November 7, 2025 11:45
Copy link
Member

@frenck frenck left a comment

Choose a reason for hiding this comment

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

@jamesonuk There seems to be a merge conflict. Can you take a look and resolve it?

Thanks 👍

../Frenck

@home-assistant home-assistant bot marked this pull request as draft November 7, 2025 11:45
@home-assistant
Copy link

home-assistant bot commented Nov 7, 2025

Please take a look at the requested changes, and use the Ready for review button when you are done, thanks 👍

Learn more about our pull request process.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the Alexa Devices integration documentation with bug fixes, new examples, and improved troubleshooting guidance.

Key Changes

  • Fixed a typo in the Amazon documentation URL (amazom.com → amazon.com)
  • Added two new examples demonstrating how to set volume and control devices in Alexa
  • Enhanced the troubleshooting section with detailed guidance for authentication errors and rate limiting issues
  • Updated the supported functionality section to include motion sensors

Comment on lines +250 to +255
You see something similar to

#### Symptom: "Not found"
- `Error retrieving devices state: Too many requests for path ['listEndpoints']`
- `Error retrieving data: CannotRetrieveData('Request failed: Bad Request')`

When trying to set up the integration, the form shows the message "Not found".
In logs.
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

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

According to the coding guidelines, lists should be surrounded by blank lines, and the sentence fragment "In logs." should be incorporated into the sentence. Consider rephrasing to: "You may see error messages similar to the following in the logs:" (combining the intro text and removing the fragment).

Copilot uses AI. Check for mistakes.

##### Description

You will see `MFA OTP code not found on login page` or `Cannot find "auth-mfa-otpcode" in html source` in the logs when trying to setup the integration. This is because the authentication details are incorrect.
Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

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

Multiple issues on this line:

  • Use "set up" (two words) when used as a verb: "trying to setup" should be "trying to set up"
  • Punctuation: There are two spaces after "integration." when only one space should be present
Suggested change
You will see `MFA OTP code not found on login page` or `Cannot find "auth-mfa-otpcode" in html source` in the logs when trying to setup the integration. This is because the authentication details are incorrect.
You will see `MFA OTP code not found on login page` or `Cannot find "auth-mfa-otpcode" in html source` in the logs when trying to set up the integration. This is because the authentication details are incorrect.

Copilot uses AI. Check for mistakes.
You will see `MFA OTP code not found on login page` or `Cannot find "auth-mfa-otpcode" in html source` in the logs when trying to setup the integration. This is because the authentication details are incorrect.

You need to ensure you are:

Copy link

Copilot AI Nov 7, 2025

Choose a reason for hiding this comment

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

According to the coding guidelines, lists should be surrounded by blank lines. There should be a blank line before the list starts.

Suggested change

Copilot uses AI. Check for mistakes.
@jamesonuk jamesonuk requested a review from frenck November 7, 2025 18:01
@jamesonuk jamesonuk marked this pull request as ready for review November 7, 2025 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants