Skip to content

[ENG-9856] create a task to periodically clean up notification sent logs#11634

Open
Ostap-Zherebetskyi wants to merge 2 commits intoCenterForOpenScience:feature/post-nr-post-releasefrom
Ostap-Zherebetskyi:feature/notifications_cleanup_task
Open

[ENG-9856] create a task to periodically clean up notification sent logs#11634
Ostap-Zherebetskyi wants to merge 2 commits intoCenterForOpenScience:feature/post-nr-post-releasefrom
Ostap-Zherebetskyi:feature/notifications_cleanup_task

Conversation

@Ostap-Zherebetskyi
Copy link
Collaborator

@Ostap-Zherebetskyi Ostap-Zherebetskyi commented Mar 12, 2026

Ticket

Purpose

create a task to periodically clean up notification sent logs

Changes

Side Effects

QE Notes

CE Notes

Documentation

@Ostap-Zherebetskyi Ostap-Zherebetskyi marked this pull request as ready for review March 12, 2026 11:25
Copy link
Collaborator

@cslzchen cslzchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Collaborator

@cslzchen cslzchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and will merge this after post-NR has been released to master.

  • Need to tweak staging settings and ask CE to test this command on staging.

@Ostap-Zherebetskyi Ostap-Zherebetskyi changed the base branch from feature/notifications-refactor-post-release to feature/post-nr-post-release March 13, 2026 14:46
@Ostap-Zherebetskyi Ostap-Zherebetskyi force-pushed the feature/notifications_cleanup_task branch from 11dab31 to 00ee1bc Compare March 13, 2026 14:53
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.

2 participants