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

Update README.md for creating the first user #2086

Open
xlorepdarkhelm opened this issue Nov 5, 2024 · 0 comments
Open

Update README.md for creating the first user #2086

xlorepdarkhelm opened this issue Nov 5, 2024 · 0 comments
Assignees

Comments

@xlorepdarkhelm
Copy link
Contributor

xlorepdarkhelm commented Nov 5, 2024

The README.md documentation needs to be updated to reflect how the system currently works, when creating the first user in a local environment. The following lines:


Creating a 'First User' in the database

After you have completed all setup steps, you will be unable to log in, because there
will not be a user in the database to link to the login.gov account you are using. So
you will need to create that user in your database using the 'create-test-user' command.

Open two terminals pointing to the api project and then run these commands in the
respective terminals.

(Server 1)
env ALLOW_EXPIRED_API_TOKEN=1 make run-flask

(Server 2)
poetry run flask command create-admin-jwt | tail -n 1 | pbcopy
poetry run flask command create-test-user --admin=True;


This is overly complicated and doesn't even work any more, since the switch to using login.gov. The documentation should be changed to:


Creating a 'First User' in the database

After you have completed all setup steps, you will be unable to log in, because there
will not be a user in the database to link to the login.gov account you are using. So
you will need to create that user in your database using the 'create-test-user' command.

Open a terminal pointing to the api project and then run this command.

poetry run flask command create-test-user --admin=True;


Since the integration into login.gov, the previous additional steps has become unnecessary, and even more, it is interfering with the ability to get prompts for the user's name, email, password, etc. and silently fails. By making this change, the first user can be created easily, and everything works as expected.

@xlorepdarkhelm xlorepdarkhelm converted this from a draft issue Nov 5, 2024
@xlorepdarkhelm xlorepdarkhelm self-assigned this Nov 7, 2024
@xlorepdarkhelm xlorepdarkhelm moved this from 🌱 New to 🏗 In progress (WIP: ≤ 3 per person) in Notify.gov product board Nov 7, 2024
@xlorepdarkhelm xlorepdarkhelm moved this from 🏗 In progress (WIP: ≤ 3 per person) to 👀 In review in Notify.gov product board Nov 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 In review
Development

No branches or pull requests

1 participant