Skip to content

Conversation

@oliverli
Copy link
Contributor

@oliverli oliverli commented Nov 10, 2025

What does this PR do? What is the motivation?

  • Updates all links in the docs from https://datadoghq.slack.com/ to https://chat.datadoghq.com/, which will allow someone to register to join the public Slack instance
    • All the usages of this URL in our public docs are only in action platform/workflow/app builder, so it doesn't go out of our domain
  • Looks like my Markdown linter also deleted a bunch of extra spaces in our docs

Merge instructions

Merge readiness:

  • Ready for merge

For Datadog employees:

Your branch name MUST follow the <name>/<description> convention and include the forward slash (/). Without this format, your pull request will not pass CI, the GitLab pipeline will not run, and you won't get a branch preview. Getting a branch preview makes it easier for us to check any issues with your PR, such as broken links.

If your branch doesn't follow this format, rename it or create a new branch and PR.

[6/5/2025] Merge queue has been disabled on the documentation repo. If you have write access to the repo, the PR has been reviewed by a Documentation team member, and all of the required checks have passed, you can use the Squash and Merge button to merge the PR. If you don't have write access, or you need help, reach out in the #documentation channel in Slack.

Additional notes

Copy link
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@oliverli oliverli marked this pull request as ready for review November 10, 2025 22:05
@oliverli oliverli requested a review from a team as a code owner November 10, 2025 22:05
@github-actions
Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

Copy link
Contributor

@iadjivon iadjivon left a comment

Choose a reason for hiding this comment

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

Hi Oliver, thanks for these changes!

I see that you made modifications in the translated versions of the docs as well! This is not needed as we have a process in place that makes these changes automatically to the non-English versions.

Would you kindly remove the changes to the non-English pages from this branch?

Once that is completed, I can go ahead and approve this for merging. Let me know if you have any questions!

@oliverli oliverli force-pushed the oliver/change-all-links-from-datadoghq.slack.com-to-chat.datadoghq.com branch from c767b99 to 2aceacd Compare November 11, 2025 20:05
@oliverli oliverli requested a review from iadjivon November 11, 2025 20:06
@oliverli oliverli closed this Nov 11, 2025
@oliverli oliverli reopened this Nov 11, 2025
@oliverli oliverli force-pushed the oliver/change-all-links-from-datadoghq.slack.com-to-chat.datadoghq.com branch from 2aceacd to 205cb9d Compare November 11, 2025 21:30
@oliverli oliverli merged commit 51358bd into master Nov 12, 2025
16 checks passed
@oliverli oliverli deleted the oliver/change-all-links-from-datadoghq.slack.com-to-chat.datadoghq.com branch November 12, 2025 05:26
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