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.

Sign up for GitHub

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

Jump to bottom

fix: Load userList only if allowed #7194

Merged

dstala merged 1 commit into develop from fix/editor-user-list

Dec 11, 2023

Member

pranavxc commented Dec 11, 2023

Change Summary

Re fix: Nested data apis(v2) permission issue #7179 (review)

Change type

feat: (new feature for the user, not a new feature for build script)
fix: (bug fix for the user, not a fix to a build script)
docs: (changes to the documentation)
style: (formatting, missing semi colons, etc; no production code change)
refactor: (refactoring production code, eg. renaming a variable)
test: (adding missing tests, refactoring tests; no production code change)
chore: (updating grunt tasks etc; no production code change)

Test/ Verification

Provide summary of changes.

Additional information / screenshots (optional)

Anything for maintainers to be made aware of


          fix: try to load userList only if allowed(based on role)

7623a44

Signed-off-by: Pranav C <pranavxc@gmail.com>

pranavxc requested a review from dstala

December 11, 2023 06:22

Contributor

github-actions bot commented Dec 11, 2023 •

edited

Uffizzi Preview deployment-42498 was deleted.

dstala approved these changes

View reviewed changes

dstala merged commit bbdfc48 into develop

22 checks passed

dstala deleted the fix/editor-user-list branch

December 11, 2023 09:11

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment