Merge lp:~maxiberta/canonical-identity-provider/fix-429-crash into lp:canonical-identity-provider/release
Proposed by
Maximiliano Bertacchini
on 2018-12-19
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Maximiliano Bertacchini on 2019-01-07 | ||||
Approved revision: | 1674 | ||||
Merge reported by: | Otto Co-Pilot | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp:~maxiberta/canonical-identity-provider/fix-429-crash | ||||
Merge into: | lp:canonical-identity-provider/release | ||||
Diff against target: |
78 lines (+26/-1) 2 files modified
src/webui/tests/test_views_registration.py (+16/-0) src/webui/views/registration.py (+10/-1) |
||||
To merge this branch: | bzr merge lp:~maxiberta/canonical-identity-provider/fix-429-crash | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Adam Collard (community) | 2018-12-19 | Approve on 2018-12-21 | |
Review via email:
|
Commit message
Show a nice error message at user registration and reset password views when the API returns 429 - too many requests.
Description of the change
- Registration: https:/
- Forgot passsword: https:/
To post a comment you must log in.
lp:~maxiberta/canonical-identity-provider/fix-429-crash
updated
on 2018-12-20
- 1674. By Maximiliano Bertacchini on 2018-12-20
-
Show the registration throttling error in the messages area.
review:
Approve