-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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 issue 19928 #20409
fix issue 19928 #20409
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #20409 +/- ##
==========================================
- Coverage 67.56% 66.36% -1.20%
==========================================
Files 991 1042 +51
Lines 109181 113132 +3951
Branches 2719 2830 +111
==========================================
+ Hits 73768 75084 +1316
- Misses 31449 33984 +2535
- Partials 3964 4064 +100
Flags with carried forward coverage won't be shown. Click here to find out more.
|
e6400b3
to
40f13ab
Compare
it needs to consider the user who is in any group that has been granted with the project admin role. Signed-off-by: wang yan <[email protected]>
Signed-off-by: wang yan <[email protected]>
Signed-off-by: wang yan <[email protected]>
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.
lgtm
Signed-off-by: wang yan <[email protected]>
Signed-off-by: wang yan <[email protected]>
Signed-off-by: wang yan <[email protected]>
it needs to consider the user who is in any group that has been granted with the project admin role.
Thank you for contributing to Harbor!
Comprehensive Summary of your change
Issue being fixed
Fixes #19928
Please indicate you've done the following: