Skip to content

Adds account lockout functionality for local accounts#142

Merged
thatarchguy merged 3 commits intomasterfrom
ft-account-lockout-106
Jul 26, 2016
Merged

Adds account lockout functionality for local accounts#142
thatarchguy merged 3 commits intomasterfrom
ft-account-lockout-106

Conversation

@mprahl
Copy link
Member

@mprahl mprahl commented Jul 21, 2016

Adds account lockout functionality for local accounts
API HTTP 400 and 404 error messages were made friendlier
Small CSS tweaks

@mprahl
Copy link
Member Author

mprahl commented Jul 21, 2016

@thatarchguy, since I modified the API spec, do you remember what you used to generate the HTML from it?

@mprahl mprahl force-pushed the ft-account-lockout-106 branch from e740400 to bb6282d Compare July 21, 2016 02:34
API HTTP 400 and 404 error messages were made friendlier
Small CSS tweaks
@mprahl mprahl force-pushed the ft-account-lockout-106 branch from bb6282d to 977a925 Compare July 21, 2016 02:36
@thatarchguy
Copy link
Member

This is what i used: https://bootprint.knappi.org

db.session.rollback()


def is_account_unlocked(username):
Copy link
Member

Choose a reason for hiding this comment

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

Would this be better on the admin model?

@mprahl
Copy link
Member Author

mprahl commented Jul 26, 2016

@thatarchguy thanks for the input. Can you take a look again?

@thatarchguy
Copy link
Member

LGTM

👍

@thatarchguy thatarchguy merged commit 8114ae9 into master Jul 26, 2016
@mprahl mprahl mentioned this pull request Jul 26, 2016
@mprahl mprahl deleted the ft-account-lockout-106 branch August 10, 2016 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants