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

org: Add slack bot integration to send timelog updates. #3021

Merged
merged 1 commit into from
Dec 1, 2024
Merged

org: Add slack bot integration to send timelog updates. #3021

merged 1 commit into from
Dec 1, 2024

Conversation

apoorvapendse
Copy link
Contributor

@apoorvapendse apoorvapendse commented Nov 30, 2024 •
edited
Loading

This PR fixes #2973.

Implmented features.

  1. Slack Oauth integration to generate and store slack bot token for organzation.
  2. UI to manage future integrations with other services.
  3. Manage command to send updates, which can be run as a cron job every hour.
  4. The messages are only sent when the UTC hour difference between current time and organization set time is <= 1hour.
  5. Organization can configure the channel they prefer to receive updates every 24 hours.
  6. Summary will contain info about all the timelogs that were dedicated for the organization is the past 24 hours.

https://github.com/user-attachments/assets/1387679c-e442-4222-89e3-5b98d46d0a70
image

@apoorvapendse apoorvapendse mentioned this pull request Nov 30, 2024
Open
@apoorvapendse apoorvapendse force-pushed the issue-2973 branch 2 times, most recently from 295aeee to d11a5b1 Compare November 30, 2024 02:29
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 30, 2024
View reviewed changes
company/views.py Fixed Show fixed Hide fixed
company/views.py Fixed Show fixed Hide fixed
github-advanced-security[bot]
github-advanced-security bot found potential problems Nov 30, 2024
View reviewed changes
company/views.py Show resolved Hide resolved
DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
.env.example Show resolved Hide resolved
DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
blt/settings.py Outdated Show resolved Hide resolved
DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
blt/urls.py Outdated Show resolved Hide resolved
DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
company/views.py Outdated Show resolved Hide resolved
DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
company/views.py Outdated Show resolved Hide resolved
DonnieBLT
DonnieBLT reviewed Dec 1, 2024
View reviewed changes
company/views.py Outdated Show resolved Hide resolved
@apoorvapendse
Copy link
Contributor Author

Thanks for the review, should we go with keeping the callback uri as environment variables?

DonnieBLT
DonnieBLT requested changes Dec 1, 2024
View reviewed changes
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.

I reviewed it and there are a couple changes, please

@apoorvapendse
Copy link
Contributor Author

I reviewed it and there are a couple changes, please

Sure I'll make them.

@apoorvapendse apoorvapendse reopened this Dec 1, 2024
@apoorvapendse apoorvapendse force-pushed the issue-2973 branch 3 times, most recently from 273fd89 to dbfa702 Compare December 1, 2024 17:11
@apoorvapendse apoorvapendse reopened this Dec 1, 2024
@DonnieBLT DonnieBLT merged commit e7885ac into OWASP-BLT:main Dec 1, 2024
7 checks passed
@DonnieBLT
Copy link
Collaborator

Thank you 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DonnieBLT DonnieBLT DonnieBLT approved these changes

Assignees
No one assigned
Labels
requested changes
Projects
None yet
Milestone
No milestone
2 participants
@apoorvapendse @DonnieBLT