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/60568 Remove permission restriction for users to set their own reminders #17642

Draft
wants to merge 2 commits into
base: release/15.2
Choose a base branch
from

Conversation

akabiru
Copy link
Member

@akabiru akabiru commented Jan 17, 2025

Ticket

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

What are you trying to accomplish?

  • Remove dedicated manage_own_reminders project permission and instead rely on view_work_packages permission - if a user can see a WP then they can create a reminder on it.
  • Ensure Anonymous users are excluded - as anonymous users can view_work_packages in some cases
  • Clean up DB of manage_own_reminders permission

Screenshots

What approach did you choose and why?

Merge checklist

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

@akabiru akabiru added this to the 15.2.x milestone Jan 17, 2025
@akabiru akabiru added the bugfix label Jan 17, 2025
@akabiru akabiru self-assigned this Jan 17, 2025
@akabiru akabiru force-pushed the bug/60568-remove-permission-restriction-for-users-to-set-their-own-reminders branch from 199c292 to 7a1065a Compare January 17, 2025 10:50
Copy link

1 Warning
⚠️ This PR has migration-related changes on a release branch. Ping @opf/operations

Generated by 🚫 Danger

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