rusackas opened a new pull request, #43901:
URL: https://github.com/apache/superset/pull/43901
### SUMMARY
#41843 (merged) added templated `{datasource_id}`, `{datasource_name}`,
`{table_names}`, and `{username}` placeholders to
`PERMISSION_INSTRUCTIONS_LINK`, so the "Request access" link on data-permission
errors can deep-link into an org's access-request tool with the denied resource
pre-filled. The security config docs only described roles/permissions in
general terms and never mentioned this. Adding a short section to the admin
security docs covering the config and its placeholders.
### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
N/A, docs-only change.
### TESTING INSTRUCTIONS
Read `docs/admin_docs/security/security.mdx` and confirm the new "Actionable
Access-Denied Messages" section (under "Managing Data Source Access for Gamma
Roles") accurately reflects the behavior in `superset/security/manager.py`'s
`_render_permission_instructions_link`.
### ADDITIONAL INFORMATION
- [ ] Has associated issue:
- [ ] Required feature flags:
- [ ] Changes UI
- [ ] Includes DB Migration (follow approval process in
[SIP-59](https://github.com/apache/superset/issues/13351))
- [ ] Migration is atomic, supports rollback & is backwards-compatible
- [ ] Confirm DB migration upgrade and downgrade tested
- [ ] Runtime estimates and downtime expectations provided
- [ ] Introduces new feature or API
- [ ] Removes existing feature or API
Co-Authored-By: Claude <[email protected]>
--
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]
