-
Notifications
You must be signed in to change notification settings - Fork 343
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
Issue #926 fixed #955
Issue #926 fixed #955
Conversation
Hello @skattel49! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2021-07-20 09:49:50 UTC |
I have changed the string formatting to f-strings as suggested. |
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.
This looks good.
Remember to run the command python manage.py makemigrations
and commit the migrations to this branch.
@skattel49, please remember to run |
These changes were merged in a separate pull request in order for me to make changes to the branch. |
Closes #926
I have fixed copied the Accounts model to ./accounts/model.py and changed it's name to Profile as instructed