Drop browserid and use allauth instead for logging in
Fixes #73 (closed)
Changes include:
- Drop browserid which only provides persona and use allauth instead.
- Drop ConfirmationProfiles and use allauth confirmation instead
- Add registration forms that use local accounts
- Add ability to change password
- Add ability to reset password
Note this merge request depends on !92 (closed) being merged in first to prevent redirect loops
@abompard I will close my other merge request !91 (closed), and split it up into pieces.
Once it's merged here I will work on getting it into hyperkitty
Merge request reports
Activity
Sure, I'll start working on it asap.
In the meantime, could you take a look at !150 (merged). I'm not sure why the tests of the original mr succeeded...
!92 (closed) is required for this branch. Otherwise we'd end up with redirect loops.Added 126 commits:
- cca8ba93...2fd81c86 - 108 commits from branch
mailman:master
- 762cfba1 - Replace browserid with allauth in setup
- f2f3864f - Update form template
- 85bc341c - Remove tests for replaces components
- e1864ee6 - Remove login template
- 859c818d - Drop AddressConfirmationProfile
- 7942575c - Remove old urls
- 09a832bd - Use new urls in base template
- cc72f091 - Update test settings
- c597b4e6 - Add links to manage account
- 454729c5 - Add app config
- 5ef0ee37 - Remove old email confirmation template
- f2fcfe2d - Update user views
- da75c062 - Add signals
- fbbbe0af - Add templates
- 6a084904 - Fix pep8 errors
- 75914234 - Fix urls
- 6cc8c8be - Rename signals and fix a small error
- 94bdb40e - Remove form to add emails from profile page
Toggle commit list- cca8ba93...2fd81c86 - 108 commits from branch
I guess I'll add a couple of tests for the signals, to make sure they work as intended. I also need to fix some existing tests. Currently I don't synchronize the addresses back from mailman. So if an address is added using any other method than postorius, it won't be picked up.
I guess we could add some way of synchronizing all addresses in the admin, this could also be exposed to the command line.
Do you think this is enough, or should we also sync the addresses whenever they can be used (probably nearly all requests)You can take a look at the templates for the emails that are sent out (signup, adding email, resetting password) and see if you would want to change any of them.
mentioned in merge request !148 (closed)
Added 24 commits:
- 4bd8589d...9d06a737 - 4 commits from branch
mailman:master
- 790d48a7 - Replace browserid with allauth in setup
- 91a486df - Update form template
- 3e7b2e14 - Remove tests for replaces components
- 3b729f20 - Remove login template
- 0f4f4e84 - Drop AddressConfirmationProfile
- 3476ceea - Remove old urls
- bd48040b - Use new urls in base template
- ce172c06 - Update test settings
- b9f934ca - Add links to manage account
- 63b3841c - Add app config
- 776020a7 - Remove old email confirmation template
- a4e589e5 - Update user views
- a5e9d325 - Add signals
- a72c1074 - Add templates
- 51614cf9 - Fix pep8 errors
- 18dbc8b0 - Fix urls
- eef3a68e - Rename signals and fix a small error
- 5b16d8e8 - Remove form to add emails from profile page
- e18a7a05 - Fix pep8 errors
- 6596b9bb - Fix login url in a test case
Toggle commit list- 4bd8589d...9d06a737 - 4 commits from branch
mentioned in merge request hyperkitty!19 (closed)