Skip to content

Update prerequisites-to-share-dashboards-charts.mdx #20739

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

Merged
merged 2 commits into from
Jun 3, 2025

Conversation

johnathannewrelic
Copy link
Contributor

Please follow conventional commit standards
in your commit messages and pull request title.

Give us some context

  • What problems does this PR solve?
  • Add any context that will help us review your changes such as testing notes,
    links to related docs, screenshots, etc.
  • If your issue relates to an existing GitHub issue, please link to it.

Copy link

netlify bot commented May 21, 2025

Deploy Preview for docs-website-netlify ready!

Name Link
🔨 Latest commit 5e76e50
🔍 Latest deploy log https://app.netlify.com/projects/docs-website-netlify/deploys/683e797b2113f9000863cc1a
😎 Deploy Preview https://deploy-preview-20739--docs-website-netlify.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.

Copy link

Hi @johnathannewrelic 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

We will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 10 to 20 minutes).

Copy link
Contributor

@cbehera-newrelic cbehera-newrelic left a comment

Choose a reason for hiding this comment

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

Please fix the suggested changes for clarity.

Each of the following personas has specific responsibilities in this process:
* **Authentication domain manager**: Controls the feature's enablement in account level.
* **Administrator**: Configures custom roles and permissions for users to manage live URLs.
The user enabling public sharing and managing user access to it can be any [user type](/docs/accounts/accounts-billing/new-relic-one-user-management/user-type/), but must be in at least one group that has Authentication Domain Manage [administation setting](/docs/accounts/accounts-billing/new-relic-one-user-management/user-management-concepts/#admin-settings) enabled.
Copy link
Contributor

Choose a reason for hiding this comment

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

Any user type can enable public sharing and manage user access for the dashboards. However, the user type must be in at least one group that has Authentication Domain Manage administation setting enabled.

Copy link
Contributor Author

@johnathannewrelic johnathannewrelic May 26, 2025

Choose a reason for hiding this comment

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

I appreciate your edit, but I would adjust it to the following to be more technically accurate: A user with any user type can enable and manage user access to public sharing . However, the user must be in at least one group that has Authentication Domain Manage administration setting enabled to do so.

@cbehera-newrelic cbehera-newrelic self-assigned this May 23, 2025
@cbehera-newrelic cbehera-newrelic added content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers) labels May 23, 2025
@adutta-newrelic
Copy link
Contributor

Hello @johnathannewrelic could you please check the suggested changes and do the needful, so that we can approve and merge this PR. Thank you

@johnathannewrelic
Copy link
Contributor Author

johnathannewrelic commented Jun 2, 2025 via email

Copy link
Contributor

@cbehera-newrelic cbehera-newrelic left a comment

Choose a reason for hiding this comment

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

LGTM

@cbehera-newrelic cbehera-newrelic merged commit 89987f9 into newrelic:develop Jun 3, 2025
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants