Skip to content

[DOCS-13416] Remove misplaced Applying restrictions section from Share Graphs page#35524

Merged
evazorro merged 2 commits intomasterfrom
eva/remove-applying-restrictions-section
Mar 25, 2026
Merged

[DOCS-13416] Remove misplaced Applying restrictions section from Share Graphs page#35524
evazorro merged 2 commits intomasterfrom
eva/remove-applying-restrictions-section

Conversation

@evazorro
Copy link
Copy Markdown
Contributor

@evazorro evazorro commented Mar 24, 2026

What does this PR do? What is the motivation?

Fixes DOCS-13416

The "Applying restrictions" section on the Share Graphs page discusses restricting access to shared dashboards by IP address. This content is about shared dashboards, not shared graphs, so I've moved it to the Shared Dashboards page.

Confirmed with @DrkSephy that this change is correct.

Merge readiness:

  • Ready for merge

Additional notes

Also cleaned up the page description and renumbered link references.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 24, 2026

2. Find your graph by using the search bar or by sorting the table columns.
3. Click on the **Revoke** button next to the graph you want to stop sharing.

## Applying restrictions
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Dear reviewer, sorry for all the link renumberings; this is the main change here. I just double-checked all the links and they did get updated correctly.

@evazorro evazorro added the WORK IN PROGRESS No review needed, it's a wip ;) label Mar 24, 2026
@evazorro evazorro removed the WORK IN PROGRESS No review needed, it's a wip ;) label Mar 24, 2026
@evazorro evazorro marked this pull request as ready for review March 24, 2026 20:32
@evazorro evazorro requested a review from a team as a code owner March 24, 2026 20:32
To revoke the keys used to share individual (embedded) graphs:

1. Navigate to [**Organization Settings > Public Sharing > Shared Graphs**][1] to see a list of all shared graphs.
1. Navigate to [**Organization Settings > Public Sharing > Shared Graphs**][2] to see a list of all shared graphs.
Copy link
Copy Markdown
Contributor

@OliviaShoup OliviaShoup Mar 24, 2026

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good checking!! I think that is probably because you're opening in in an org where you don't have access to that setting. I had to switch to my personal sandbox org to see it.

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.

OH that makes sense! thank you :)

@OliviaShoup OliviaShoup self-requested a review March 24, 2026 22:08
Copy link
Copy Markdown
Contributor

@OliviaShoup OliviaShoup left a comment

Choose a reason for hiding this comment

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

🚀

@evazorro evazorro merged commit d230cc5 into master Mar 25, 2026
32 of 38 checks passed
@evazorro evazorro deleted the eva/remove-applying-restrictions-section branch March 25, 2026 15:29
StefonSimmons pushed a commit that referenced this pull request Apr 16, 2026
…e Graphs page (#35524)

* [DOCS-13416] Remove misplaced Applying restrictions section from Share Graphs page

* Add IP address restrictions section to Shared Dashboards page
genesor pushed a commit that referenced this pull request Apr 19, 2026
…e Graphs page (#35524)

* [DOCS-13416] Remove misplaced Applying restrictions section from Share Graphs page

* Add IP address restrictions section to Shared Dashboards page
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