-
Notifications
You must be signed in to change notification settings - Fork 2
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
Permission Set Group Issue #39
Comments
Hello @grahampcurran, could you please run:
Where "USERID" is the ID of the user you are trying to open the report for. Please censor anything confidential. |
And then if you run:
This will allow me to see what data the code is working with. |
Looks like the 3rd from bottom is the issue, I should probably add an additional condition to filter it out.
Sorry I keep asking you to run queries, this should be the last one. I added 2 more fields and I expected PermissionSet.Type or Permissionset.IsOwnedByProfile to be usable on filtering out the "(null PermissionSet)" |
Looks like this is due to a muting permission set in the permission set group. In the query it shows up as an empty PermissionSet.Name. I have added a fix and will put it in the 1.2.1 release. |
Thanks! That confirms my suspicion, it's the muting permission set that causes the trouble, I've patched it and include it in a hotfix release. |
@grahampcurran I appreciate it has been nearly a year, but I've released a fix for this now haha. I'm going to close this as fixed. |
Hi there,
I am getting the following error when trying to run the permissions report in Production:
Uncaught (in promise) TypeError: Cannot read property 'Name' of null at B.getPermissionSetNames (build.js:2) at async i.runReport (build.js:2) at chrome-extension://gkibdicghcpcikhjpgjoijpobdipklnp/public/async chrome-extension:/gkibdicghcpcikhjpgjoijpobdipklnp/build.js:2
The User has permission sets and a permission set group assigned.
The text was updated successfully, but these errors were encountered: