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

Show only updated time, not actor if first notification is date alert #16472

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

oliverguenther
Copy link
Member

@oliverguenther oliverguenther commented Aug 19, 2024

Ticket

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

What are you trying to accomplish?

When the first/newest notification is a date alert, we should only show the timestamp, as that notification does not have an actor.

Merge checklist

  • Added/updated tests
  • Tested major browsers (Chrome, Firefox, Edge, ...)

@HDinger HDinger merged commit 392a74b into dev Aug 19, 2024
12 checks passed
@HDinger HDinger deleted the fix/notifications-date-alert branch August 19, 2024 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants