-
Notifications
You must be signed in to change notification settings - Fork 1.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #13382: Replace the user ID with user name in Glossary Term #13753
Fixes #13382: Replace the user ID with user name in Glossary Term #13753
Conversation
Hi there 👋 Thanks for your contribution! The OpenMetadata team will review the PR shortly! Once it has been labeled as Let us know if you need any help! |
Hi @NormanZhang-zzt Thanks for your contribution. Can you catch displayName using existing method which will use already fetched data for user details if not it will query via API
|
Also can you attach screenshot of your changes will be helpful to review your PR much faster Thanks. |
Quality Gate passed for 'open-metadata-ui'The SonarCloud Quality Gate passed, but some issues were introduced. 2 New issues |
Describe your changes:
Fixes #13382
I worked on replacing the UserId with the User Name because displaying user name could be more intuitive and useful
Type of change:
Checklist:
Fixes <issue-number>: <short explanation>