-
Notifications
You must be signed in to change notification settings - Fork 83
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
feat: integrate cloud event for webhook #213
Conversation
@goharbor/all-maintainers Please help to review this proposal, thanks! |
0356872
to
eeedc09
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.
The pyload format name should be correctly set to "CloudEvents"
eeedc09
to
c452280
Compare
c452280
to
c5b748a
Compare
Signed-off-by: chlins <chenyuzh@vmware.com>
c5b748a
to
281cde4
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.
lgtm
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
@Vad1mo Hi, the comments have been resolved, please help to review again, thanks. |
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.
🎉
That's fantastic folks!!!!!!!! |
Link: goharbor/harbor#18209
Signed-off-by: chlins chenyuzh@vmware.com