goingforstudying-ctrl commented on PR #40746: URL: https://github.com/apache/superset/pull/40746#issuecomment-4636024368
Pushed an update to the `_extract_role_names` validator: - Bare strings are now rejected instead of being split into characters. - Empty role lists are preserved as `[]` rather than collapsed to `None`. - `DetachedInstanceError` on `Role.name` is caught per-item and skipped. - Added unit tests covering string rejection, empty-list preservation, role-object coercion, detached-instance handling, and the `serialize_user_object` -> `UserInfo` round trip. Let me know if anything else looks off. -- 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]
