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.

Sign up for GitHub

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

Jump to bottom

Set is_staff to true for transit agency staff users #2242

Closed
1 task
Tracked by #2261
angela-tran opened this issue Jul 23, 2024 · 1 comment · Fixed by #2315
Closed
1 task
Tracked by #2261

Set is_staff to true for transit agency staff users #2242

angela-tran opened this issue Jul 23, 2024 · 1 comment · Fixed by #2315
Assignees
@machikoyasuda
Milestone
Benefits admin to...

Comments

@angela-tran
Copy link
Member

angela-tran commented Jul 23, 2024 •
edited
Loading

In order to access the admin site, a user must have is_staff set to true.

Any user who can authenticate with an agency domain (via *_ALLOWABLE_DOMAIN env var) can log in (has the experience of a user that is not in any group, can do nothing) - i.e. is set to is_staff=True

Let's make sure to have unit tests covering this logic.

Acceptance Criteria

  • A user in the list is able to login with the currently supported SSO
@angela-tran angela-tran converted this from a draft issue Jul 23, 2024
@angela-tran angela-tran mentioned this issue Jul 31, 2024
Closed
16 tasks
@angela-tran angela-tran changed the title Transit agency staff should be able to login Set is_staff to true for transit agency staff users Aug 1, 2024
@angela-tran angela-tran moved this from Needs shaping to Todo in Digital Services Aug 1, 2024
@angela-tran angela-tran moved this from Todo to Needs shaping in Digital Services Aug 6, 2024
@angela-tran angela-tran moved this from Needs shaping to Todo in Digital Services Aug 8, 2024
@angela-tran
Copy link
Member Author

This ticket needs some dev team discussion to agree on how the list is maintained.

@machikoyasuda machikoyasuda self-assigned this Aug 14, 2024
@machikoyasuda machikoyasuda mentioned this issue Aug 21, 2024
Merged
@machikoyasuda machikoyasuda moved this from Todo to In progress in Digital Services Aug 21, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Digital Services Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@machikoyasuda machikoyasuda

Labels
None yet
Projects
Digital Services
Status: Done
Milestone
Benefits admin tool (agency users)
Development

Successfully merging a pull request may close this issue.

Admin: Test that transit user does not get added to calitp group cal-itp/benefits
2 participants
@machikoyasuda @angela-tran