Skip to content
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

Fix(UI) #8555: Tags removal issue #9055

Merged
merged 13 commits into from
Nov 30, 2022
Merged

Fix(UI) #8555: Tags removal issue #9055

merged 13 commits into from
Nov 30, 2022

Conversation

aniketkatkar97
Copy link
Member

@aniketkatkar97 aniketkatkar97 commented Nov 29, 2022

Describe your changes :

I worked on the following:

  • Fixed Error, thrown while removing tags from the data asset details page
  • Fixed bug where users were not able to remove the tags on the Dashboard Details page
  • Changes in MlModelFeaturesList component:
    • Tag removal bug fix
    • Removed tailwind use and replaced with antd
    • Code optimization
    • Localisation changes

Type of change :

  • Bug fix
  • Improvement

Frontend Preview (Screenshots) :

Table Details Page:

Screen.Recording.2022-11-29.at.7.12.56.PM.mov

Dashboard Details Page:

Screen.Recording.2022-11-29.at.7.13.29.PM.mov

Ml Model Details Page:

Screen.Recording.2022-11-29.at.7.13.58.PM.mov

SourceList component:

Screen.Recording.2022-11-30.at.9.58.09.AM.mov

Checklist:

  • I have read the CONTRIBUTING document.
  • I have commented on my code, particularly in hard-to-understand areas.
  • I have added tests that prove my fix is effective or that my feature works.
  • All new and existing tests passed.

Reviewers

Frontend: @open-metadata/ui

@aniketkatkar97 aniketkatkar97 added the UI UI specific issues label Nov 29, 2022
@aniketkatkar97 aniketkatkar97 requested a review from a team November 29, 2022 13:46
@aniketkatkar97 aniketkatkar97 self-assigned this Nov 29, 2022
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:46 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:46 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:46 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:47 Inactive
@github-actions github-actions bot added the safe to test Add this label to run secure Github workflows on PRs label Nov 29, 2022
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:47 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:47 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:47 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 13:47 Inactive
@cypress
Copy link

cypress bot commented Nov 29, 2022



Test summary

175 0 9 0


Run details

Project openmetadata
Status Passed
Commit 5206d80
Started Nov 30, 2022 2:35 PM
Ended Nov 30, 2022 3:10 PM
Duration 34:40 💡
OS Linux Ubuntu - 22.04
Browser Chrome 107

View run in Cypress Dashboard ➡️


This comment has been generated by cypress-bot as a result of this project's GitHub integration settings. You can manage this integration in this project's settings in the Cypress Dashboard

@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 14:44 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 14:44 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 14:44 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 14:44 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 16:51 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 29, 2022 16:51 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 06:25 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 06:25 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 11:57 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 11:57 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:12 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:12 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:13 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:13 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:22 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:22 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:22 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 12:22 Inactive
@aniketkatkar97 aniketkatkar97 enabled auto-merge (squash) November 30, 2022 12:41
@sonarqubecloud
Copy link

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

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

47.6% 47.6% Coverage
0.0% 0.0% Duplication

@aniketkatkar97 aniketkatkar97 added UI UI specific issues and removed UI UI specific issues labels Nov 30, 2022
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 13:33 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 13:33 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 13:34 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 13:34 Inactive
@sonarqubecloud
Copy link

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

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 14:15 Inactive
@aniketkatkar97 aniketkatkar97 temporarily deployed to cypress November 30, 2022 14:15 Inactive
@sonarqubecloud
Copy link

[OpenMetadata-Platform] Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
safe to test Add this label to run secure Github workflows on PRs UI UI specific issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants