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

feat: notification to analysts when upcoming milestone is due #3598

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

AntBush
Copy link
Contributor

@AntBush AntBush commented Oct 3, 2024

Implements NDT-362

  • Check to trigger automatic release process

  • Check for automatic rebasing

@AntBush AntBush changed the title NDT-362-notifications-to-analysts-when-milestone-reports-are-coming-due feat: notification to analysts when upcoming milestone is due Oct 3, 2024
@AntBush AntBush force-pushed the NDT-362-notifications-to-analysts-when-milestone-reports-are-coming-due branch from a52907c to 4e1be19 Compare October 3, 2024 15:09
app/backend/lib/milestoneDueDate.ts Fixed Show fixed Hide fixed
app/backend/lib/milestoneDueDate.ts Fixed Show fixed Hide fixed
app/tests/backend/lib/milestoneDueDate.test.ts Dismissed Show dismissed Hide dismissed
@AntBush AntBush force-pushed the NDT-362-notifications-to-analysts-when-milestone-reports-are-coming-due branch from 4e1be19 to e26ac22 Compare October 3, 2024 15:27
@@ -89,6 +90,7 @@
try {
const token = await getAccessToken();
const emailResult = await sendEmailMerge(token, body, subject, contexts);
console.log(emailResult, contexts, details);

Check warning

Code scanning / ESLint

Disallow the use of `console` Warning

Unexpected console statement.
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.

3 participants