Archives/gristlabs_grist-core
1
0
Fork 0
mirror of https://github.com/gristlabs/grist-core.git synced 2024-10-27 20:44:07 +00:00
Code Issues Projects Releases Wiki Activity

(core) Hide id column in Access Rules

Browse Source 
Summary: The id column was a possible option for rulesets, which was a bug.

Test Plan: Browser tests.

Reviewers: paulfitz

Reviewed By: paulfitz

Subscribers: paulfitz

Differential Revision: https://phab.getgrist.com/D3817
This commit is contained in:
George Gevoian 2023-03-13 22:03:55 -04:00
parent efd92c6c2a
commit 8a6962d3e6
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
app/client/aclui/AccessRules.ts
View File

@ -507,7 +507,8 @@ export class AccessRules extends Disposable {
// Returns a list of valid colIds for the given table, or undefined if the table isn't valid.
public getValidColIds(tableId: string): string[]|undefined {
return this._aclResources.get(tableId)?.colIds.filter(id => !isHiddenCol(id)).sort();
return this._aclResources.get(tableId)?.colIds.filter(id =>
!isHiddenCol(id) && id !== 'id').sort();
}
// Get rules to use for seeding any new set of table/column rules, e.g. to give owners