Skip to content

Fix Developer Platform link in OAuth2 in Datadog doc#37484

Open
holdenmc wants to merge 2 commits into
masterfrom
holden/fix-oauth2-publishing-platform-link
Open

Fix Developer Platform link in OAuth2 in Datadog doc#37484
holdenmc wants to merge 2 commits into
masterfrom
holden/fix-oauth2-publishing-platform-link

Conversation

@holdenmc

Copy link
Copy Markdown

What does this PR do? What is the motivation?

Updates the "Developer Platform" link in the "Implement the OAuth protocol" section from https://app.datadoghq.com/apps to https://app.datadoghq.com/publishing-platform.

Note: There is a second occurrence of https://app.datadoghq.com/apps on this page — it appears as the anchor text for "SHA-256" in the Code Challenge Method table of the "Use the PKCE extension" section. That link appears to be a dead end / incorrect target, but I'm not sure what it should point to instead. Leaving it for a follow-up.

Merge instructions

Merge readiness:

  • Ready for merge

Additional notes

@holdenmc holdenmc marked this pull request as ready for review June 12, 2026 17:02
@holdenmc holdenmc requested a review from a team as a code owner June 12, 2026 17:02
@github-actions

Copy link
Copy Markdown
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@evazorro evazorro self-assigned this Jun 12, 2026
@evazorro

Copy link
Copy Markdown
Contributor

Note: There is a second occurrence of https://app.datadoghq.com/apps on this page — it appears as the anchor text for "SHA-256" in the Code Challenge Method table of the "Use the PKCE extension" section. That link appears to be a dead end / incorrect target, but I'm not sure what it should point to instead. Leaving it for a follow-up.

@holdenmc yeah, it looks like that link was introduced when the doc was first created, but I agree that is probably not what it should point to. I'm just going to remove the link, because SHA-256 is an easy enough thing to google; I don't think we need to link to an article about it or anything.

@holdenmc

Copy link
Copy Markdown
Author

@evazorro agreed, that makes sense!

@evazorro evazorro left a comment

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.

Thank you for fixing this!

@evazorro evazorro removed their assignment Jun 12, 2026
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.

2 participants