Skip to content

Update SAML-enabling docs language #18500

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

RajatBist
Copy link

they don't need support to enable SAML for them (there is no feature flag, etc on our end), they should have the resources to do this on their own

What does this PR do?

Edits the language of the doc to clarify any confusion around enabling SAML (and whether Datadog support has to step in to complete this)

Motivation

https://datadog.zendesk.com/agent/tickets/1233142

Additional Notes


Reviewer checklist

  • Review the changed files.
  • Review the URLs listed in the Preview section.
  • Check images for PII
  • Review any mentions of "Contact Datadog support" for internal support documentation.

they don't need support to enable SAML for them (there is no feature flag, etc  on our end), they should have the resources to do this on their own
@RajatBist RajatBist requested a review from a team as a code owner June 12, 2023 22:08
@RajatBist RajatBist changed the title Update _index.md Update SAML-enabling docs language Jun 12, 2023
Copy link
Contributor

@rtrieu rtrieu left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks for this update!

@rtrieu
Copy link
Contributor

rtrieu commented Nov 7, 2024

/merge

@dd-devflow
Copy link

dd-devflow bot commented Nov 7, 2024

Devflow running: /merge

View all feedbacks in Devflow UI.


2024-11-07 19:09:30 UTC ℹ️ MergeQueue: waiting for PR to be ready

This merge request is not mergeable yet, because of pending checks/missing approvals. It will be added to the queue as soon as checks pass and/or get approvals.
Note: if you pushed new commits since the last approval, you may need additional approval.
You can remove it from the waiting list with /remove command.


2024-11-07 23:09:33 UTC ⚠️ MergeQueue: This merge request was unqueued

This merge request was unqueued

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants