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

getCurrentUserEffectivePermissions low-privileged users fix #1079

Merged
merged 1 commit into from
Mar 31, 2020
Merged

getCurrentUserEffectivePermissions low-privileged users fix #1079

merged 1 commit into from
Mar 31, 2020

Conversation

koltyakov
Copy link
Member

Category

  • Bug fix?
  • New feature?
  • New sample?
  • Documentation update?

Related Issues

fixes #259, this fix was previously applied to v1 but got lost in v2, so literally it's a "cherry-pick" of #846

What's in this Pull Request?

Changes the way of requesting EffectiveBasePermissions for low-privilege users by consuming /EffectiveBasePermissions endpoint except for getting user effective permissions by a login.

@koltyakov koltyakov modified the milestones: 2.0.4, 2.0.5 Feb 27, 2020
@patrick-rodgers patrick-rodgers modified the milestones: 2.0.5, 2.0.4 Mar 31, 2020
@patrick-rodgers
Copy link
Member

Awesome work as always, thanks!

@patrick-rodgers patrick-rodgers merged commit e14b59a into pnp:version-2 Mar 31, 2020
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

getCurrentUserEffectivePermissions Access Denied for user with read access
2 participants