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

Fix Manager Login #8

Merged
merged 4 commits into from Jun 27, 2022
Merged

Fix Manager Login #8

merged 4 commits into from Jun 27, 2022

Conversation

PEMIDI
Copy link

@PEMIDI PEMIDI commented Jun 14, 2022 •
edited

When the manager want to login, must have type exactly username in lowercase or uppercase characters. This is an issue for who wants to login.

That's what I did to fix the problem.:

  • Lowercase the usernames of manager in the database.
  • Change View.py to save username only in lowercase in database, after getting it from login form.

Also Two little issues were fixed:

  • Last migration was not created, so I created it and added it to git.
  • Account/decorators.py has an indent mistake.

@AminAliH47 AminAliH47 merged commit c0a92ba into AminAliH47:develop Jun 27, 2022
@AminAliH47
Copy link
Owner

Thank you for your cooperation, Your pull request was merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@PEMIDI @AminAliH47