[#5820] fix invite user with existing email error#5880
Conversation
|
This one looks good @amercader @wardi @mutantsan!
|
|
This looks great @mutantsan ,can you fix the conflicts so I can merge? Cheers |
|
Sorry @mutantsan still some failures |
|
@mutantsan I tried to fix the failures as it looked like a straight forward missing var but I'm still getting some failures after fixing that: My changes: https://github.com/ckan/ckan/compare/mutantsan-5820-invite-user-with-existing-email |
5042731 to
338f08a
Compare
|
Sorry, @amercader , that's my fault. I had to fix it immediately. |
|
@mutantsan I re-run the tests and they passed: https://app.circleci.com/pipelines/github/ckan/ckan/2810/workflows/d0cb749a-9b79-4d7c-af32-b70e34daff04 If you fix the conflicts I'll merge this and backport it |
|
This was merged to master already (5708fc2), I've backported it to 2.9 |
Fix for issue #5820
I've added validators and catch the ValidationError.
Features:
Please [X] all the boxes above that apply