Skip to content

[ENG-9857] Refactor notification type references to use NotificationTypeEnum#11520

Open
Ostap-Zherebetskyi wants to merge 3 commits intoCenterForOpenScience:feature/notifications-refactor-post-releasefrom
Ostap-Zherebetskyi:fix/consolidate_notification_type_enum
Open

[ENG-9857] Refactor notification type references to use NotificationTypeEnum#11520
Ostap-Zherebetskyi wants to merge 3 commits intoCenterForOpenScience:feature/notifications-refactor-post-releasefrom
Ostap-Zherebetskyi:fix/consolidate_notification_type_enum

Conversation

@Ostap-Zherebetskyi
Copy link
Collaborator

Purpose

Refactor notification type references to use NotificationTypeEnum

  • Updated all instances of NotificationType.Type to NotificationTypeEnum in test files and application code.
  • Ensured consistency in notification type usage.
  • This change improves clarity and maintainability by standardizing the notification type references.

Changes

See diff

QA Notes

N/A

Documentation

N/A

Side Effects

N/A

Ticket

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

@Ostap-Zherebetskyi Ostap-Zherebetskyi marked this pull request as ready for review January 2, 2026 15:18
- Updated all instances of NotificationType.Type to NotificationTypeEnum in test files and application code.
- Ensured consistency in notification type usage across various tests including auth, claims, events, and more.
- This change improves clarity and maintainability by standardizing the notification type references.
@Ostap-Zherebetskyi Ostap-Zherebetskyi force-pushed the fix/consolidate_notification_type_enum branch from c313970 to bc5afe6 Compare February 3, 2026 10:39
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.

1 participant