aaron-wolmutt commented on PR #58292: URL: https://github.com/apache/airflow/pull/58292#issuecomment-3578845076
> > I'm going to make @jason810496 requested changes and wait for more feedback. I think we can include non-token information about the user in a ui_atrributes object (so ui_attributes fields will vary based on which auth manager is used). There is an issue open to display the username right now, so the endpoint is a convenience, and I don't think it violates the separation of concerns. > > If the purpose of this PR is to resolve the enable/disable button, then I think we should wait until the discussion in issue come up with final schema with the new meta permision router intead of the current implementation, and maybe close this one and raise a new PR for it. > > Or if the purpose of this PR is just for showing "Who I am" on the bottom-left setting tab, then I think we can skip it, but it will serve as differnt purpose than #57966. We should just use it as a convenience for the "who i am" and close it out if that's ok -- 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]
