Re: [PR] fix: #33862 – Security menu missing on MySQL due to case mismatch [superset]

["rusackas (via GitHub)" <gi...@apache.org>]

rusackas commented on PR #33864:
URL: https://github.com/apache/superset/pull/33864#issuecomment-3644067111

   Hi @caerux 👋
   
   This PR has been on hold for about 6 months. Per the discussion, this change needs a DB migration to avoid breaking existing deployments that have roles attached to the current permission names.
   
   @mistercrunch asked if there are reusable constructs/methods to easily generate permission renames as part of DB migrations - that might be a good starting point.
   
   Are you still interested in working on this? If so, adding a migration would unblock the PR. If you need help or guidance on the migration approach, feel free to ask!
   
   Thanks for your contribution!


-- 
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: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org