dgarciabriseno commented on PR #36025:
URL: https://github.com/apache/superset/pull/36025#issuecomment-3534283062

   Thanks for verifying this. I'll need to double check my permissions when I'm 
back at work. 
   
   I have a 5.0.0 instance running that I demo'd this morning by opening a 
private browsing window and it worked fine.
   
   On November 14, 2025 7:05:14 PM UTC, Sam Firke ***@***.***> wrote:
   >@sfirke requested changes on this pull request.
   >
   >I like this idea but I don't think this set of permissions is sufficient. I 
created a new role with these seven permissions, then gave a dashboard that 
role and tried to view it in a private browsing window where I wasn't signed 
in. It didn't work - I was redirected to login.
   >
   >I have a functional Public role that I've used for several years. It has 31 
permissions. I wonder if not all 31 are needed and the minimal set is between 
yours and mine?
   >
   >I'm happy to re-review if you can expand this to a set such that once that 
role is added, I can view the dashboard while logged out.
   >
   >
   >
   >-- 
   >Reply to this email directly or view it on GitHub:
   >https://github.com/apache/superset/pull/36025#pullrequestreview-3466391311
   >You are receiving this because you authored the thread.
   >
   >Message ID: ***@***.***>


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to