Skip to content

DOCS-676 - App permissions update #5080

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 12 commits into from
Apr 28, 2025
Merged

DOCS-676 - App permissions update #5080

merged 12 commits into from
Apr 28, 2025

Conversation

jpipkin1
Copy link
Collaborator

Purpose of this pull request

This pull request adds a release note and updates this article: https://help.sumologic.com/docs/manage/content-sharing/

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

DOCS-676

@jpipkin1 jpipkin1 added doc:update Revisions and updates to content do-not-merge Pull requests that should not be merged labels Feb 14, 2025
@jpipkin1 jpipkin1 self-assigned this Feb 14, 2025
@cla-bot cla-bot bot added the cla-signed Contributor approved, listed in .clabot file label Feb 14, 2025
@jpipkin1 jpipkin1 changed the title Docs 676 - App permissions update DOCS-676 - App permissions update Apr 15, 2025
Copy link

@sjain05 sjain05 left a comment

Choose a reason for hiding this comment

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

edit and manage access is not allowed for installed apps in content library

one screenshot needs to be changed as it shows Grant Edit and View Access as this should not be the case

@sjain05
Copy link

sjain05 commented Apr 22, 2025

you can read more about the requirements here https://sumologic.atlassian.net/browse/SUMO-178257

@jpipkin1
Copy link
Collaborator Author

Thanks for the review, @sjain05! I resolved your comments above by adding a note:

App content can only be shared with view access. Edit and manage access is not allowed for installed apps.

You also said:

one screenshot needs to be changed as it shows Grant Edit and View Access as this should not be the case

The screenshot you mentioned is not for an app, but for a dashboard, so it doesn't need to be changed:
PermissionsShare

@jpipkin1 jpipkin1 requested a review from sjain05 April 22, 2025 18:55
@sjain05
Copy link

sjain05 commented Apr 24, 2025

Member

beware that you took screenshot of a dashboard that is part of an app Acquia and we don't give edit access on dashboards, searches or other content which is part of an app

@jpipkin1
Copy link
Collaborator Author

Thanks, @sjain05! I recaptured the screenshot. It should be good now.
PermissionsShare

@sjain05
Copy link

sjain05 commented Apr 28, 2025

@jpipkin1 can we merge the PR? feature is rolled out to prods

@jpipkin1 jpipkin1 enabled auto-merge April 28, 2025 18:46
@jpipkin1 jpipkin1 added this pull request to the merge queue Apr 28, 2025
Merged via the queue into main with commit 46dd3fd Apr 28, 2025
5 checks passed
@jpipkin1 jpipkin1 deleted the docs-676-app-permissions branch April 28, 2025 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged doc:update Revisions and updates to content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants