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: Appstate cleanup #13765

Merged

chirag-madlani merged 1 commit into main from appstate-cleanup

Oct 30, 2023

Contributor

karanh37 commented Oct 30, 2023

Describe your changes:

Cleanup of Appstate

I worked on ... because ...

Type of change:

Checklist:

I have read the CONTRIBUTING document.
My PR title is Fixes <issue-number>: <short explanation>
I have commented on my code, particularly in hard-to-understand areas.
For JSON Schema changes: I updated the migration scripts or explained why it is not needed.


          fix: remove usage of current user from appstate

ea46706

karanh37 requested review from harshach and chirag-madlani as code owners

October 30, 2023 08:09

github-actions bot added UI safe to test labels

chirag-madlani approved these changes

View reviewed changes

Contributor

github-actions bot commented Oct 30, 2023

Jest test Coverage

UI tests summary

Lines	Statements	Branches	Functions
	48.63% (23233/47778)	30.36% (8659/28520)	30.14% (2639/8757)

sonarqubecloud bot commented Oct 30, 2023

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

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
1.2% Duplication

chirag-madlani merged commit 4fbb538 into main

10 of 13 checks passed

chirag-madlani deleted the appstate-cleanup branch

October 30, 2023 10:24

Ashish8689 pushed a commit that referenced this pull request


          fix: remove usage of current user from appstate (#13765)

962a8ec

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

Labels

safe to test UI