@sgiehl opened this Pull Request on June 22nd 2022 Member

Description:

While trying to fix #18750 a couple of other issue within the UserManager popped up. Guess some of them might have regressed during the vue migration.

Note: This PR also adds a proper handling of bulk requests to AjaxHandler. This is e.g. important when setting a role/capability for multiple users at once. The requests are sent as bulk. Currently it would not show any error if some of the requests failed.

fixes #18750

Review

@sgiehl commented on July 12th 2022 Member

This one should be ready for review now. I've already tried to explain some of the changes in some comments above. Hope that makes it easier to understand everything.

This Pull Request was closed on July 20th 2022
Powered by GitHub Issue Mirror