-
Notifications
You must be signed in to change notification settings - Fork 9
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
Feat: transit processor portal link for admin dashboard #2370
Conversation
5823a62
to
931274e
Compare
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would it be possible to bake some of these setup steps into the fixtures file?
E.g. creating a transit agency user and group, linking everything together, adding a sample portal link...
552e016
to
cd854a6
Compare
Ah good idea, cd854a6 should simplify the setup |
This is great! But I'd still like to shortcut the user setup. I created a local user account and exported the JSON, can you please add this to the fixture here? {
"model": "auth.user",
"pk": 3,
"fields": {
"password": "pbkdf2_sha256$870000$5pS1ll33O6bi2rCCELW8kZ$OvFJJIERCEE0H2RhI8WzKTi+tFhbrfo/sxFdwAUzESI=",
"last_login": null,
"is_superuser": false,
"username": "cst-user",
"first_name": "CST",
"last_name": "User",
"email": "[email protected]",
"is_staff": true,
"is_active": true,
"date_joined": "2024-09-17T18:54:46Z",
"groups": [2],
"user_permissions": []
}
} The account details are stored in a shared LastPass item: I also think it would be most helpful if our default value (in |
- set up a sample agency user using the fixtures file - show form on the admin page to simplify logging in with sample users
I was wondering how to streamline it further and this is great, thanks! The changes are in eba2877 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fantastic, thank you for addressing my feedback!
Closes #2268
This PR adds a button that links to the
TransitProcessor
's portal so that a user can open the portal in a new tab.To test
python manage.py migrate
local_fixtures.json
if neededUsing the admin panelInGroups
add aCST
groupInTransit Agencys
addCST
to theStaff group
andCustomer service group
fieldsInTransit processors
addhttps://control.littlepay.com
to thePortal url
fieldInUsers
addCST
to yourChosen groups
and remove theSuperuser status
property from your userPortal url
field to blank and verify that theTransit Processor
section does not appear in the home dashboard