Skip to content
This repository has been archived by the owner on May 13, 2024. It is now read-only.

feat(notifications): project budget check notifications #901

Merged
merged 1 commit into from
Dec 7, 2022

Conversation

trowik
Copy link
Member

@trowik trowik commented Nov 8, 2022

Created a management command to check the current budget of a project. The new command creates a note in the corresponding Redmine issue if the budget exceeded 30% or 70%.

@trowik trowik force-pushed the budget_check_cronjob branch 2 times, most recently from 0a72695 to 196f7f9 Compare November 8, 2022 07:41
@trowik trowik force-pushed the budget_check_cronjob branch 5 times, most recently from 27e6c70 to 816d906 Compare November 22, 2022 16:16
@trowik trowik force-pushed the budget_check_cronjob branch 2 times, most recently from af72e4c to 293c3a8 Compare November 30, 2022 15:22
Created a management command to check the current budget of a project.
The new command creates a note in the corresponding Redmine issue if the budget
exceeded 30% or 70%.
@trowik trowik force-pushed the budget_check_cronjob branch from 293c3a8 to b81e28e Compare December 6, 2022 10:28
@trowik trowik marked this pull request as ready for review December 6, 2022 12:06
@trowik trowik requested a review from a team as a code owner December 6, 2022 12:06
@trowik trowik requested a review from fugal-dy December 6, 2022 12:06
@trowik
Copy link
Member Author

trowik commented Dec 6, 2022

@fugal-dy we already did review this PR, however I've made some minor changes since then.

Copy link
Contributor

@fugal-dy fugal-dy left a comment

Choose a reason for hiding this comment

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

pair reviewed

good job.

@trowik trowik merged commit 293757d into adfinis:master Dec 7, 2022
@trowik trowik deleted the budget_check_cronjob branch December 14, 2022 08:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants