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

Email Reminders and Dashboard Updates #591

Open
wants to merge 21 commits into
base: master
Choose a base branch
from

Conversation

miknevinas
Copy link
Contributor

@miknevinas miknevinas commented Mar 14, 2024

This PR is pretty hefty. I'll try to summarize all the changes as neatly as possible:

  1. The 'Sharing' view is overridden to do a couple of things:
  • Send an email to a user (via a celery task) assigned as a 'Reviewer' on a particular page/item
  • Add an optional 'Message' field that appends additional text to the email
  • Display a warning if the user is operating in a directory with a lot of items, which will cause a short delay when granting permissions. The system recursion limit is also temporarily increased to accomplish this (Note: this functionality was taken from a separate PR which can be closed: Sharing permissions fix #584)
  1. A system is added to send reminder emails to users about their assigned pages
  • On initial assignment, the user is immediately sent an email via a celery task
  • Users are sent a reminder email via a CRON job if they have not removed themselves as a reviewer within 5 days
  • Both of these leverage the cache to track assigned items and their due dates, along with any custom messages
  • Only 'Reviewer' roles are tracked throughout this process, though this can be changed if requested
  1. Three new columns are added to the dashboard to display items that the logged-in user has modified, created or been assigned
  • The 'modified' and 'assigned' columns require their own respective indexes to render, which are added in both the default and upgrade profiles
  • All columns can now be hidden if desired
  • The icons and verbiage have been removed as requested, though specifics on the new layout were never specified.

***NOTE:

The only way I could properly register the indexes was by rebuilding the catalog in Site Setup after running the upgrade. I tried both reindexing and re-cataloging programmatically in the upgrade profile, but neither of these worked. I'm open to ideas of how to fix this.

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