Skip to content

[ENG-9923] Sync naming of notifications#11611

Merged
cslzchen merged 3 commits intoCenterForOpenScience:feature/notifications-refactor-post-releasefrom
bodintsov:feature/clean-up-and-sync-type-defenitions
Feb 27, 2026
Merged

[ENG-9923] Sync naming of notifications#11611
cslzchen merged 3 commits intoCenterForOpenScience:feature/notifications-refactor-post-releasefrom
bodintsov:feature/clean-up-and-sync-type-defenitions

Conversation

@bodintsov
Copy link
Contributor

Ticket

https://openscience.atlassian.net/browse/ENG-9923

Purpose

Remove duplicated naming, unify existing

Changes

Remove duplicated naming, unify existing

Side Effects

TBD

QE Notes

TBD

CE Notes

TBD

Documentation

TBD

Copy link
Collaborator

@Ostap-Zherebetskyi Ostap-Zherebetskyi left a comment

Choose a reason for hiding this comment

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

LGTM ⭐

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.

'user_username': user.username,
}
)
# TODO: Should ask PO whether we should send USER_REQUEST_DEACTIVATION mail
Copy link
Collaborator

Choose a reason for hiding this comment

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

Let's follow up with PO after review.

@cslzchen cslzchen merged commit 0003dc8 into CenterForOpenScience:feature/notifications-refactor-post-release Feb 27, 2026
8 checks passed
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.

3 participants