Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timing updates for rendering and preparation of user group granular permissions details to resolve intermittent error on race condition #17877

Open
wants to merge 2 commits into
base: v15/dev
Choose a base branch
from

Conversation

AndyButland
Copy link
Contributor

Prerequisites

  • I have added steps to test this contribution in the description below

Fixes #17860, tracked under AB 47493 (internal HQ tracker).

Description

This was tricky to replicate as it doesn't happen all the time, but I could fairly consistently find that after five or six refreshes of a user group detail page open in the backoffice - i.e. /umbraco/section/user-management/workspace/user-group/edit/{id} - you would see what's reported in the linked issue (no granular permission details displayed, and the error in the console).

With the updates in this PR in place I could no longer see the issue.

To Test:

  • Other than visual code inspection that the updates make sense, refresh and navigate multiple times to the user group detail and ensure the granular permissions component renders correctly and without console errors.
  • Verify granular permissions can be set for a user group.

…ermissions details to resolve intermittent error on race condition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant