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

Chore: add fixture for Cal-ITP Staff account #2677

Merged
merged 1 commit into from
Feb 11, 2025
Merged

Conversation

lalver1
Copy link
Member

@lalver1 lalver1 commented Feb 10, 2025

Closes #2675

This PR adds to the database a calitp-user to make local testing and QA a little simpler without having to manually set up user accounts.

Reviewing

Reset your database using the new fixture file in this PR, log in to the admin area, confirm that you can use the calitp-user credentials that are stored in the shared LastPass area, and that the permissions match the Cal-ITP staff group.

@lalver1 lalver1 self-assigned this Feb 10, 2025
@github-actions github-actions bot added migrations [auto] Review for potential model changes/needed data migrations updates deployment-dev [auto] Changes that will trigger a deploy if merged to dev labels Feb 10, 2025
Copy link

Coverage report

This PR does not seem to contain any modification to coverable code.

@lalver1 lalver1 force-pushed the chore/add-staff-account branch from 098a2d9 to bfd0ba9 Compare February 10, 2025 20:59
@lalver1 lalver1 force-pushed the chore/add-staff-account branch from bfd0ba9 to d00380e Compare February 10, 2025 22:22
@lalver1 lalver1 marked this pull request as ready for review February 10, 2025 22:29
@lalver1 lalver1 requested a review from a team as a code owner February 10, 2025 22:29
Copy link
Member

@angela-tran angela-tran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for setting this up. Worked for me 👍

@lalver1 lalver1 merged commit afa910f into main Feb 11, 2025
9 checks passed
@lalver1 lalver1 deleted the chore/add-staff-account branch February 11, 2025 00:29
@thekaveman
Copy link
Member

thekaveman commented Feb 11, 2025

Just a reminder (mostly for myself, I forgot!) the Cal-ITP staff group with id=1 (referenced by this Cal-ITP staff account) is created by the migrations: https://github.com/cal-itp/benefits/blob/main/benefits/core/migrations/0013_create_staff_group.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment-dev [auto] Changes that will trigger a deploy if merged to dev migrations [auto] Review for potential model changes/needed data migrations updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add fixture for Cal-ITP Staff account
3 participants