This repository was archived by the owner on Jan 15, 2022. It is now read-only.
PLF-8447 : "Send me a digest email" on notification settings is not w… #599
Conversation
This file contains hidden or 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
…orking (#412) * PLF-8447 : Resume Daily and weekly digest mails jobs (#406) * PLF-8447 : Resume Daily and weekly digest mails jobs This commit add an upgrade plugin to resume jobs in environments in which the migration is already done. * PLF-8447 : Resume Daily and weekly digest mails jobs Before this fix, the Upgrade Plugin was not launched if the platform is already in version 6.0 In fact, for tribe, this plugin will be not launched. The fix change the target version for the UP so that it will be started also on 6.0.x * PLF-8447 : Send Digest mail are not working Before this fix, when the Upgrade Plugin is applied after a lot af weeks without running, all pending notification will be treated and sent. This Fix remove all daily and weekly notifications which are pending. * Use MailNotificationStorage instead of DAO * PLF-8447 : manage error. If an error occurs, throw it to relaunch the UP at next startup * PLF-8447 : limit the digest to version after 5.2.0 * PLF-8447 : the delete digest must be run if oldVersion is 5.2.0 * Update exception management * Add unit test * Update file headers
boubaker
approved these changes
Nov 11, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
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.
…orking (#412)
PLF-8447 : Resume Daily and weekly digest mails jobs (PLF-5407 : CLONE - Improve performances by not using OrganizationService #406)
PLF-8447 : Resume Daily and weekly digest mails jobs
This commit add an upgrade plugin to resume jobs in environments in which the migration is already done.
Before this fix, the Upgrade Plugin was not launched if the platform is already in version 6.0
In fact, for tribe, this plugin will be not launched.
The fix change the target version for the UP so that it will be started also on 6.0.x
PLF-8447 : Send Digest mail are not working
Before this fix, when the Upgrade Plugin is applied after a lot af weeks without running,
all pending notification will be treated and sent.
This Fix remove all daily and weekly notifications which are pending.
Use MailNotificationStorage instead of DAO
PLF-8447 : manage error. If an error occurs, throw it to relaunch the UP at next startup
PLF-8447 : limit the digest to version after 5.2.0
PLF-8447 : the delete digest must be run if oldVersion is 5.2.0
Update exception management
Add unit test
Update file headers