Skip to content
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

Add tooltip to remind webhook slack rate limit #18507

Closed
chlins opened this issue Apr 10, 2023 · 0 comments · Fixed by #18787
Closed

Add tooltip to remind webhook slack rate limit #18507

chlins opened this issue Apr 10, 2023 · 0 comments · Fixed by #18787
Assignees
@AllForNothing
Labels
area/ui area/webhook target/2.9.0

Comments

@chlins
Copy link
Member

chlins commented Apr 10, 2023

How can we help you?

Add the tooltip to remind user that the slack has the rate limit when harbor sends webhooks to slack. As described in the slack document(https://api.slack.com/docs/rate-limits) no more than 1 request per second.
AllForNothing pushed a commit to AllForNothing/harbor that referenced this issue Jun 5, 2023
Add a tooltip for slack notification
e2ec37d 
1.Fixes goharbor#18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
@AllForNothing AllForNothing mentioned this issue Jun 5, 2023
Merged
5 tasks
AllForNothing pushed a commit to AllForNothing/harbor that referenced this issue Jun 5, 2023
Add a tooltip for slack notification
5e5d189 
1.Fixes goharbor#18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
AllForNothing pushed a commit to AllForNothing/harbor that referenced this issue Jun 5, 2023
@AllForNothing
Add a tooltip for slack notification
90a504a 
1.Fixes goharbor#18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
AllForNothing pushed a commit to AllForNothing/harbor that referenced this issue Jun 6, 2023
@AllForNothing
Add a tooltip for slack notification
0d79168 
1.Fixes goharbor#18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
wy65701436 pushed a commit that referenced this issue Jun 7, 2023
@AllForNothing
Add a tooltip for slack notification (#18787)
2f51daf 
1.Fixes #18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
WilfredAlmeida pushed a commit to WilfredAlmeida/harbor that referenced this issue Jul 8, 2023
@AllForNothing @WilfredAlmeida
Add a tooltip for slack notification (goharbor#18787)
0546e32 
1.Fixes goharbor#18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
Signed-off-by: Wilfred Almeida <60785452+WilfredAlmeida@users.noreply.github.com>
WilfredAlmeida pushed a commit to WilfredAlmeida/harbor that referenced this issue Jul 8, 2023
@AllForNothing @WilfredAlmeida
Add a tooltip for slack notification (goharbor#18787)
e46f6e9 
1.Fixes goharbor#18507
2.Remind the users of Slack's rate limits

Signed-off-by: AllForNothing <sshijun@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AllForNothing AllForNothing

Labels
area/ui area/webhook target/2.9.0
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add a tooltip for slack notification AllForNothing/harbor
3 participants
@wy65701436 @AllForNothing @chlins