-
Notifications
You must be signed in to change notification settings - Fork 175
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
Update for the system robot account and the project robot account sections #517
Conversation
@Vad1mo @OrlinVasilev Please help review the English text, thanks!!! |
…tions 1. Fixes goharbor/harbor#19354 Signed-off-by: AllForNothing <shijun.sun@broadcom.com>
3d9aef2
to
57630cd
Compare
Vadim, because we have to GA Harbor 2.10.0, before raise another PR to update the current comments, I merge the current PR
Hi, @Vad1mo @OrlinVasilev I 'll merge the current PR and raise another PR to fix issues mentioned in your comments. |
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
fixes issues mentioned in PR goharbor#517 Signed-off-by: stonezdj <stonezdj@gmail.com>
This is how the
Permission References
tables look like: