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.

Sign up for GitHub

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

Jump to bottom

DIAC-835 Filter notifications sent in the last 7 days #2459

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

DIAC-835 Filter notifications sent in the last 7 days #2459

wants to merge 12 commits into from
+172 −25

Conversation

alivenichoppa
Copy link
Contributor

@alivenichoppa alivenichoppa commented Jan 21, 2025 •
edited
Loading

Jira link

https://tools.hmcts.net/jira/browse/DIAC-835

Change description

Save Notifications only if they are sent in the last 7 days.
Add configuration to schedule saveNotificationsToData event to specified time.

Checklist

  • commit messages are meaningful and follow good commit message guidelines
  • README and other documentation has been updated / added (if needed)
  • tests have been updated / new tests has been added (if needed)
  • Does this PR introduce a breaking change

@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 24, 2025 17:30 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 24, 2025 18:29 Abandoned
@hmcts-jenkins-d-to-i hmcts-jenkins-d-to-i bot requested a deployment to preview January 24, 2025 18:57 Abandoned
@alivenichoppa alivenichoppa marked this pull request as ready for review January 28, 2025 09:40
KleoG
KleoG previously approved these changes Jan 28, 2025
View reviewed changes
oleon23
oleon23 previously approved these changes Jan 28, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2025

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Feb 3, 2025
@github-actions github-actions bot removed the stale label Feb 4, 2025
@alivenichoppa alivenichoppa dismissed stale reviews from oleon23 and KleoG via a8178ec February 5, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@oleon23 oleon23 oleon23 left review comments

@KleoG KleoG KleoG left review comments

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
enable_keep_helm enable_timed_event_service ns:ia prd:ia rel:ia-case-api-pr-2459
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@alivenichoppa @KleoG @oleon23