Skip to content

Search EEEU - Hide Limited Access #337

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

Merged
Gunjan Datta (gudatta) merged 3 commits into from
Aug 22, 2026
Merged

Search EEEU - Hide Limited Access #337

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
a1ff88d
Updated lib to fix issue w/ column rendering.
gudatta Aug 20, 2026
52fbe9d
Added the object type to the search item. Added limited access filter.
gudatta Aug 21, 2026
73b4c81
Fixed sorting issue w/ permissions table.
gudatta Aug 22, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,10 +19,10 @@
"adm-zip": "^0.5.18",
"datatables.net": "^3.0.0",
"datatables.net-bs5": "^3.0.0",
"dattatable": "^2.11.83",
"dattatable": "^2.11.87",
"exceljs": "^4.4.0",
"fast-xml-parser": "^5.9.3",
"gd-sprest-bs": "^10.15.91",
"gd-sprest-bs": "^10.15.92",
"jszip": "^3.10.1",
"mammoth": "^1.12.0",
"moment": "^2.30.1",
Expand Down
2 changes: 1 addition & 1 deletion spfx/config/package-solution.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"solution": {
"name": "site-admin-tool",
"id": "9c7bcdac-024d-4ffe-b63a-cef526bc1930",
"version": "0.9.9.2",
"version": "0.9.9.3",
"includeClientSideAssets": true,
"skipFeatureDeployment": true,
"isDomainIsolated": false,
Expand Down
20 changes: 20 additions & 0 deletions src/reports/permissions.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -558,9 +558,14 @@ export class Permissions {
name: "IsLimitedAccess",
title: "Is<br/>Limited Access?"
},
/** These need to be tested with sorting the value is sorting as a string not number. */
{
name: "",
title: "M365<br/>Groups",
onClickHeader: (el) => {
// Set the data type
el.setAttribute("data-type", "num");
},
onRenderCell: (el, col, item: IPermissionItem) => {
// Set the value
el.setAttribute("data-order", item.GroupIds.length.toString());
Expand All @@ -570,6 +575,10 @@ export class Permissions {
{
name: "",
title: "M365<br/>Users",
onClickHeader: (el) => {
// Set the data type
el.setAttribute("data-type", "num");
},
onRenderCell: (el, col, item: IPermissionItem) => {
// Set the value
el.setAttribute("data-order", item.GroupUsersCount.toString());
Expand All @@ -579,6 +588,10 @@ export class Permissions {
{
name: "",
title: "AD<br/>Accounts",
onClickHeader: (el) => {
// Set the data type
el.setAttribute("data-type", "num");
},
onRenderCell: (el, col, item: IPermissionItem) => {
// Ensure the site members exist
if (item.SiteMembers) {
Expand All @@ -603,7 +616,14 @@ export class Permissions {
{
name: "",
title: "Site<br/>Users",
onClickHeader: (el) => {
// Set the data type
el.setAttribute("data-type", "num");
},
onRenderCell: (el, col, item: IPermissionItem) => {
// Set the data type
el.setAttribute("data-type", "num");

if (item.Type == "User") {
el.innerHTML = "1";
el.setAttribute("data-order", "1");
Expand Down
Loading
Loading