-
Notifications
You must be signed in to change notification settings - Fork 44
fix: issues in New Frontend #400
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
Conversation
8077c98
to
9f26f13
Compare
…te Project button
bea61ad
to
a0b2581
Compare
a0b2581
to
2cfdc54
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go
@duonglq-tsdv Besides i appreciate the fixes, would be more sensible if you do one pull request for fixes. |
Thank you for the feedback! I’ll make sure to follow this in the future, even for minor changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
Pull Request Description
What does this PR do?
Pre-condition:
How to Test?
1. [Delete Project] Enabled the Delete button.
Pre-condition:
Procedure:
Expected:
Note:
2. [Project List] Fixed the behaviors of the Clearing Request icon.
Pre-condition:
Procedure:
Expected:
3. [Edit & View Release] Corrected data in "License Scan Report Result," "Legal Evaluation," and "License Agreement" fields
Pre-condition:
Procedure:
+ "License Scan Report Result"
+ "Legal Evaluation"
+ "License Agreement"
Expected:
4. [Request] Prevent application from signing out when visiting the Request page
Procedure:
Expected:
Note: