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.

Sign up for GitHub

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

Jump to bottom

Fix #3051: Glossary-term suggest api has fqdn same as name #3052

Merged

harshach merged 1 commit into main from issue-3051

Mar 1, 2022

Collaborator

harshach commented Mar 1, 2022

Describe your changes :

Type of change :

Bug fix

Frontend Preview (Screenshots) :

For frontend related change, please link screenshots of your changes preview! Optional for backend related changes.

Checklist:

I have read the CONTRIBUTING document.
I have performed a self-review of my own.
I have tagged my reviewers below.
I have commented on my code, particularly in hard-to-understand areas.
My changes generate no new warnings.
I have added tests that prove my fix is effective or that my feature works.
All new and existing tests passed.

Reviewers


          Fix #3051: Glossary-term suggest api has fqdn same as name

a5b6de4

darth-coder00 approved these changes

View reviewed changes

sonarcloud bot commented Mar 1, 2022

[open-metadata-ingestion] Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

mithmatt approved these changes

View reviewed changes

sonarcloud bot commented Mar 1, 2022

[catalog] Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

harshach merged commit 0b78978 into main

harshach deleted the issue-3051 branch

March 1, 2022 19:19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment