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.

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

Bounty Bot Enhancement: Fixed Label Updates & Slack Integration #66

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

rishyym0927
Copy link

πŸ›  Changes Introduced:

βœ… Implemented bounty tracking to update the existing bounty label instead of adding duplicates.
βœ… Improved GitHub issue comments to update the existing bounty message instead of creating multiple comments.
βœ… Sponsorship history tracking: Each user’s total sponsored developers count is tracked and displayed.
βœ… Replaced SLACK_WEBHOOK_URL with SLACK_BOT_TOKEN for better Slack integration.
βœ… Improved Slack notifications to reflect the updated bounty amount.
πŸ”₯ How It Works:

A user comments /bounty $X on an issue.
The bot updates the issue label to reflect the total bounty.
The bot updates the GitHub comment instead of adding duplicate messages.
The bot tracks the number of developers the user has sponsored.
A Slack message is sent to notify about the bounty.

πŸ” Testing Done:

βœ” Verified that bounties are correctly updated.
βœ” Tested Slack notifications for accuracy.
βœ” Ensured the GitHub comment updates correctly.

###Changes Made
-You need to manually add the SLACK_BOT_TOKEN and PERSONAL_ACCESS_TOKENS in secrets under actions

###Screenshots

image
image

###FIXES OWASP-BLT/BLT#3767

Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not correct.

@rishyym0927
Copy link
Author

Hey I have reverted those changes and made fixes

@rishyym0927 rishyym0927 requested a review from DonnieBLT March 5, 2025 07:42
Copy link
Collaborator

@DonnieBLT DonnieBLT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This also needs to add the label to the issue and add a comment on who did the /bounty command - also we don't need the blt-action.yml we can just update the readme to add the slack bot token

@rishyym0927
Copy link
Author

@DonnieBLT I have removed the action files and shifted it to readme.md. Also the label are automatically added on commenting /bounty with the total amount.

image

image

@rishyym0927 rishyym0927 requested a review from DonnieBLT March 7, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Action & Slackbot: /bounty $X Command
2 participants