-
Notifications
You must be signed in to change notification settings - Fork 1
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
Son/techn 373 apply template for email notifier #102
Open
nsonanh
wants to merge
29
commits into
son/techn-371-fix-todo-tasks-discovery-scope
Choose a base branch
from
son/techn-373-apply-template-for-email-notifier
base: son/techn-371-fix-todo-tasks-discovery-scope
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Son/techn 373 apply template for email notifier #102
nsonanh
wants to merge
29
commits into
son/techn-371-fix-todo-tasks-discovery-scope
from
son/techn-373-apply-template-for-email-notifier
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
nsonanh
commented
Dec 30, 2022
- [@dhealthdapps/backend] feat(notifier): add & use templates for alert & report email notifications
- [@dhealthdapps/backend] test(notifier): update & fix failed unit tests
- [@dhealthdapps/backend] refactor(notifier): remove DappHelper dependency in AlertNotifier
nsonanh
force-pushed
the
son/techn-371-fix-todo-tasks-discovery-scope
branch
from
January 13, 2023 08:49
ad6c39a
to
244c935
Compare
nsonanh
force-pushed
the
son/techn-373-apply-template-for-email-notifier
branch
from
January 13, 2023 09:02
835d017
to
d0eed7a
Compare
- @dhealth/contracts@0.6.1 - @dhealth/components@0.6.1 - @dhealthdapps/backend@0.6.1 - @dhealthdapps/frontend@0.6.1
…ration & update classes to use it instead of direct loader calls
…stead of the actual content in StateDTO
…ifier scope's dependency
…ized headers and reconstruct when decode
…scheduler to query from configurable sources instead of main app's public key
…e of discoverTransactions scheduler to be false if app is in staging/production environment
… & report email notifications
…ncy in AlertNotifier
…prevent same content notifications repeatedly
… classes and let strava classes extend them
…vent handler to allow different providers
…hTotal() & create & use model.find() for find() in QueryService
merged in main |
evias
force-pushed
the
son/techn-373-apply-template-for-email-notifier
branch
from
January 17, 2023 12:47
d0eed7a
to
aa90d5a
Compare
vega-hit
approved these changes
Jul 3, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.