Skip to content

Conversation

gitstevenpham
Copy link
Contributor

@gitstevenpham gitstevenpham commented Aug 16, 2025

https://docs-staging.datadoghq.com/steven.pham/dsm/dsm-messages-permission-setup/data_streams/messages/#creating-a-new-role-with-permissions

What does this PR do? What is the motivation?

What changed

Improved the user experience of the role creation documentation in the Data Streams Monitoring Messages setup guide. The changes make the permission setup process more accessible and easier to follow.

Why it changed

The original role creation instructions were technical and difficult to follow, especially for users who may not be familiar with Datadog's permission system. Users needed clearer, step-by-step guidance to successfully create roles and assign permissions for the Messages feature.

Where the changes occurred

  • content/en/data_streams/messages.md - Enhanced the "Creating a new role with permissions" section

How it affects the system

This is purely a documentation improvement that doesn't affect any system behavior. It provides:

  • Better structured step-by-step instructions
  • Clear visual formatting with bold UI elements
  • Helpful callout boxes warning users about permission requirements
  • More descriptive language and examples
  • Fixed duplicate link references

The changes make it easier for users to set up the required permissions for Data Streams Monitoring Messages feature.

Merge instructions

Merge readiness:

  • Ready for merge

Copy link
Contributor

github-actions bot commented Aug 16, 2025

✅ Documentation Team Review

The documentation team has approved this pull request. Thank you for your contribution!

Copy link
Contributor

Preview links (active after the build_preview check completes)

Modified Files

@gitstevenpham gitstevenpham changed the title Improve DSM Messages role creation guide clarity DSMON-988 Improve DSM Messages role creation guide clarity Aug 16, 2025
@gitstevenpham gitstevenpham marked this pull request as ready for review August 19, 2025 17:19
@gitstevenpham gitstevenpham requested review from a team as code owners August 19, 2025 17:19
Copy link
Contributor

@janine-c janine-c left a comment

Choose a reason for hiding this comment

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

Hey Steven, this looks great! I made some small readability suggestions.

The one thing I want to flag is that this kind of documentation can pose an additional maintenance burden: we already have general documentation on how to create roles and assign them, and if either workflow changes, those pages are the most likely to be updated, and pages that duplicate those workflows can get missed.

If you have a good reason to duplicate it, it's not the end of the world by any means to have it, but I wanted to make sure we at least thought about whether it would be worth it first!

gitstevenpham and others added 3 commits August 19, 2025 16:19
Co-authored-by: Janine Chan <64388808+janine-c@users.noreply.github.com>
Co-authored-by: Janine Chan <64388808+janine-c@users.noreply.github.com>
Co-authored-by: Janine Chan <64388808+janine-c@users.noreply.github.com>
@gitstevenpham
Copy link
Contributor Author

@janine-c Thanks for the feedback and suggestions! Totally hear you on the duplication risk. We did consider that, but felt it was important to step things out explicitly here to guide users through this specific flow. We’ll be sure to keep an eye on the related documentation if things change.

Copy link
Contributor

@janine-c janine-c left a comment

Choose a reason for hiding this comment

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

Thanks for the additional context, @gitstevenpham! As long as it's on your radar, I'm happy to approve 🎉

@gitstevenpham gitstevenpham merged commit 0a775d1 into master Aug 19, 2025
17 of 18 checks passed
@gitstevenpham gitstevenpham deleted the steven.pham/dsm/dsm-messages-permission-setup branch August 19, 2025 21:00
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