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

As sys admin, I need the Access Request emails to admins to resume #2687

Closed
5 of 6 tasks
ADPennington opened this issue Aug 30, 2023 · 2 comments · Fixed by #3050
Closed
5 of 6 tasks

As sys admin, I need the Access Request emails to admins to resume #2687

ADPennington opened this issue Aug 30, 2023 · 2 comments · Fixed by #3050
Assignees
Labels
dev raft review This issue is ready for raft review techdebt

Comments

@ADPennington
Copy link
Collaborator

ADPennington commented Aug 30, 2023

Description:

When we transitioned from IronPort to SendGrid (#2613), emails to admins on the number of users that have requested access (#2490) stopped. It looks like this is one email trigger we missed.

Acceptance Criteria:

  • OFA System Admin and OFA Admin receives one email per day (ideal run time between 0100-0200 UTC) presenting a count of all users with an "Access Request" status as of that day. Prototype in notes.
  • Django Admin console has additional filter to show timestamp of when a user's status changed to, "Access request"
  • Testing Checklist has been run and all tests pass
  • README is updated, if necessary

Tasks:

  • Investigate why our cronjob is not working
  • Run Testing Checklist and confirm all tests pass

Notes:
You can recreate this locally or in dev environments by messing with cron here and troubleshooting it manually. Could also use Flower or Redis CLI to debug further.
Screenshot 2024-05-28 at 3.12.13 PM.png

@robgendron robgendron added techdebt Refined Ticket has been refined at the backlog refinement labels Apr 23, 2024
@robgendron robgendron removed the Refined Ticket has been refined at the backlog refinement label May 28, 2024
@robgendron
Copy link

@andrew-jameson and @raftmsohani will refine tasks offline.

@andrew-jameson andrew-jameson changed the title As sys admin I need the Access Request emails to admins to resume As sys admin, I need the Access Request emails to admins to resume May 28, 2024
@robgendron robgendron added the raft review This issue is ready for raft review label Jul 1, 2024
@raftmsohani
Copy link

Note: discussed this AC: Django Admin console has additional filter to show timestamp of when a user's status changed to, "Access request" with @ADPennington and we decided this AC is not needed.
The Admin can always sort the user based on access date, and additionally the date range filter is not looking nice.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev raft review This issue is ready for raft review techdebt
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants