Skip to content

Make it clearer that users have to install OAuth credentials for nest integration setup#41793

Merged
frenck merged 2 commits into
home-assistant:currentfrom
ryanmahan:patch-1
Nov 22, 2025
Merged

Make it clearer that users have to install OAuth credentials for nest integration setup#41793
frenck merged 2 commits into
home-assistant:currentfrom
ryanmahan:patch-1

Conversation

@ryanmahan
Copy link
Copy Markdown
Contributor

Clarify instructions for adding OAuth credentials in Home Assistant to push users towards installing them

Proposed change

The language for installing the applications credentials could be stronger here. The current language reads like this step and the link to Application Credentials is informational. This PR changes the language to make it clear that there is an action the user needs to take.

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.

Clarify instructions for adding OAuth credentials in Home Assistant to push users towards installing them
@home-assistant home-assistant Bot added the current This PR goes into the current branch label Nov 15, 2025
@netlify
Copy link
Copy Markdown

netlify Bot commented Nov 15, 2025

Deploy Preview for home-assistant-docs ready!

Name Link
🔨 Latest commit 3f4d45e
🔍 Latest deploy log https://app.netlify.com/projects/home-assistant-docs/deploys/6921aabbdd4792000893dc10
😎 Deploy Preview https://deploy-preview-41793--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.

Comment thread source/_integrations/nest.markdown Outdated
![Screenshot of OAuth Client ID and Client Secret](/images/integrations/nest/oauth_created.png)

8. You now have *OAuth Client ID* and *OAuth Client Secret* needed by Home Assistant. See [Application Credentials](/integrations/application_credentials) for more general detail about how Home Assistant manages credentials.
8. You now have the *OAuth Client ID* and *OAuth Client Secret* needed by Home Assistant. Follow the [instructions for Application Credentials](/integrations/application_credentials) to add the *OAuth Client ID* and *OAuth Client Secret* in Home Assisstant.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The reason the wording is this way is because you are asked for these credentials during integration setup, so its not necessary to do additional step to add them as you'll be asked for them. I agree with your point here, though perhaps we should say that you can add them manually or add them when the integration asks you for them?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sounds like my confusion here was just my own user error. I had an old token set up in application credentials for nest that I believe had expired. Didn't see a prompt during the set up to input the client secret and missed that it needed to be replaced

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Well it definitely can be confusing because it doesn't ask you again if the credentials are already setup. It just uses them, assuming you want to use them and home assistant can't tell if there is an error with them.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I agree, I also think the current wording suggested doesn't harm.

So, let's move forward on getting this PR into a resoluted state 👍

../Frenck

Comment thread source/_integrations/nest.markdown Outdated
Copy link
Copy Markdown
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.

Thanks, @ryanmahan 👍

../Frenck

                       

Blogging my personal ramblings at frenck.dev

@frenck frenck merged commit 4b56956 into home-assistant:current Nov 22, 2025
7 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

current This PR goes into the current branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants