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.

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

feat: reactivate natural sorting order of notification by status #779

Conversation

poojapatel23
Copy link
Contributor

@poojapatel23 poojapatel23 commented Mar 21, 2024

This PR introduces a new feature: reactivate natural sorting order of notifications by status

[Issue]:

[Summary]:

  • This commit provides the sorting order of notifications (alerts and investigations) to follow a natural sorting order based on status. Previously, the sorting method was sorted alphabetically

[Changes Made]:

  • Modified/Added the SQL query responsible for sorting notifications by status.

[Context/Reasoning]:

  • Natural sorting by status provides a more intuitive and user-friendly display of notifications.

ds-mmaul and others added 30 commits March 15, 2024 16:25
…oauth-client-scope

Feature/698 oauth client scope
ds-mwesener and others added 26 commits April 2, 2024 15:04
…x-cypress

chore(feature): 616 - Cypress test fix.
…x-notification-flow

Chore/616 fix notification flow
…e-helm-release-1.3.33

Prepare Helm release for next version
feature: 616 Allow edc notification update with empty asset list, release 10.8.0
…e-helm-release-1.3.34

Prepare Helm release for next version
…repare-helm-release-1.3.34

Prepare Helm release for next version
…tatus

[Summary]:
- This commit provides the sorting order of notifications (alerts and investigations) to follow a natural sorting order based on status. Previously, the sorting method was sorted alphabetically

[Changes Made]:
- Modified/Added the SQL query responsible for sorting notifications by status.

[Context/Reasoning]:
- Natural sorting by status provides a more intuitive and user-friendly display of notifications.
…natural-order' into feature/status-according-to-the-natural-order

# Conflicts:
#	tx-backend/src/main/java/org/eclipse/tractusx/traceability/notification/infrastructure/notification/model/NotificationMessageBaseEntity.java
@poojapatel23 poojapatel23 marked this pull request as draft April 4, 2024 13:56
@poojapatel23 poojapatel23 deleted the feature/status-according-to-the-natural-order branch April 4, 2024 14:02
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.

6 participants