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

bug/60449 Reminders generate two or more lines for a work package in notification center, clicking one selects all #17611

Conversation

akabiru
Copy link
Member

@akabiru akabiru commented Jan 15, 2025

Ticket

https://community.openproject.org/projects/communicator-stream/work_packages/60449/activity#activity-9

What are you trying to accomplish?

Aggregate the Reminder notifications with other notifications (regular aggregation) so that only one item per work package is visible in the notification center.

Screenshots

Screenshot 2025-01-15 at 1 07 00 PM Screenshot 2025-01-15 at 1 23 56 PM

What approach did you choose and why?

This reverts: #17426 where we previously defined reminder notifications as standalone.

Merge checklist

  • Added/updated tests
  • Added/updated documentation in Lookbook (patterns, previews, etc)
  • Tested major browsers (Chrome, Firefox, Edge, ...)

…on center, clicking one selects all

https://community.openproject.org/work_packages/60449

Aggregate the Reminder notifications with other notifications (regular aggregation) so that
only one item per work package is visible in the notification center.

This reverts: #17426 where we previously defined reminder notifications
as standalone.
@akabiru akabiru added the bugfix label Jan 15, 2025
@akabiru akabiru added this to the 15.2.x milestone Jan 15, 2025
@akabiru akabiru self-assigned this Jan 15, 2025
When there exists a reminder + date alert for a given WP, show the date
alert text + the reminder note
@akabiru akabiru closed this Jan 15, 2025
@akabiru akabiru deleted the bug/60449-reminders-generate-two-or-more-lines-for-a-work-package-in-notification-center,-clicking-one-selects-all branch January 15, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

1 participant