-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature: Email notifcations for qtrly report status changes #2764
The head ref may contain hidden characters: "2729\u2043QR-Email-notifications"
Conversation
The lint errors are from a recent formatting change affecting the entire codebase #2763 If you just rebase and lint it should grab them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@CarsonF - On to you.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Progress @atGit2021! 💪
src/components/progress-report/workflow/progress-report-workflow.service.ts
Outdated
Show resolved
Hide resolved
src/components/progress-report/workflow/progress-report-workflow.service.ts
Outdated
Show resolved
Hide resolved
src/components/progress-report/workflow/progress-report-workflow.service.ts
Outdated
Show resolved
Hide resolved
src/components/progress-report/workflow/progress-report-workflow.service.ts
Outdated
Show resolved
Hide resolved
src/components/progress-report/workflow/progress-report-workflow.service.ts
Outdated
Show resolved
Hide resolved
src/core/email/templates/progress-report-status-changed.template.tsx
Outdated
Show resolved
Hide resolved
src/core/email/templates/progress-report-status-changed.template.tsx
Outdated
Show resolved
Hide resolved
src/core/email/templates/progress-report-status-changed.template.tsx
Outdated
Show resolved
Hide resolved
src/components/progress-report/workflow/progress-report-workflow.repository.ts
Outdated
Show resolved
Hide resolved
src/components/progress-report/workflow/progress-report-workflow.repository.ts
Outdated
Show resolved
Hide resolved
fee6be3
to
6663fa6
Compare
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Those changes look good, sir!
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
...omponents/progress-report/workflow/handlers/progress-report-workflow-notification.handler.ts
Outdated
Show resolved
Hide resolved
ee51619
to
507bef6
Compare
e4a1c64
to
12867ca
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great @atGit2021! I just did some last 10% polish. This was a huge task, great job 🎉
12867ca
to
8a83b3c
Compare
┆Issue is synchronized with this Monday item by Unito