Skip to content

Commit

Permalink
Merge pull request #2790 from bcameron1231/roleDef-fix
Browse files Browse the repository at this point in the history
Update security.md doc for Role Def
  • Loading branch information
bcameron1231 authored Sep 26, 2023
2 parents e29b6a9 + ee81e9e commit be251da
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/sp/security.md
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,7 @@ const user = await sp.web.currentUser();
const r = await list.roleAssignments.add(user.Id, defs[0].Id);

// remove a role assignment
const { Id: fullRoleDefId } = await list.roleDefinitions.getByName('Full Control')();
const { Id: fullRoleDefId } = await sp.web.roleDefinitions.getByName('Full Control')();
const ras = await list.roleAssignments();
// filter/find the role assignment you want to remove
// here we just grab the first
Expand Down

0 comments on commit be251da

Please sign in to comment.