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

feat(permissions): filter fields in forms based on read/write permissions #2180

Merged
Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
420e45f
feat(permissions): filter fields in forms based on read/write permiss…
tomwwinter Jan 15, 2024
0e627bc
fix: reset demo-permission-generator
tomwwinter Jan 15, 2024
ab8a9d6
refactor: disable-wrapper component for better Cognitive Complexity s…
tomwwinter Jan 15, 2024
efa5e04
Merge branch 'master' into tw/feat/1912-filter-fields-if-user-has-no-…
tomwwinter Jan 15, 2024
020d3bb
fix: use correct import path in ability.service.ts
tomwwinter Jan 15, 2024
e5301eb
refactor: move filter logic to entity-form component/service
tomwwinter Jan 15, 2024
38f8c81
fix: typo
tomwwinter Jan 15, 2024
41d779c
test: improve test coverage
tomwwinter Jan 15, 2024
ed6181e
Merge branch 'master' into tw/feat/1912-filter-fields-if-user-has-no-…
tomwwinter Jan 15, 2024
5248fe0
fix: test syntax
tomwwinter Jan 16, 2024
f99b8d0
fix: test syntax
tomwwinter Jan 16, 2024
754bf3e
fix: pr review feedback
tomwwinter Jan 16, 2024
cf79872
fix: typo
tomwwinter Jan 16, 2024
b1feafc
fix: typo
tomwwinter Jan 16, 2024
c2290c8
fix: pr review feedback
tomwwinter Jan 16, 2024
6e5a4f2
Merge branch 'master' into tw/feat/1912-filter-fields-if-user-has-no-…
tomwwinter Jan 18, 2024
b692321
fix: remove unnecessary function call
tomwwinter Jan 18, 2024
8142222
fix: code format
tomwwinter Jan 18, 2024
dd2f5ff
fix: code format
tomwwinter Jan 18, 2024
4988f07
Merge branch 'master' into tw/feat/1912-filter-fields-if-user-has-no-…
TheSlimvReal Jan 22, 2024
baecf55
disabled check also happens on form initialization
TheSlimvReal Jan 22, 2024
14e58a9
Merge branch 'master' into tw/feat/1912-filter-fields-if-user-has-no-…
tomwwinter Jan 24, 2024
429428d
fix: use create permissions when form creates new entity
tomwwinter Jan 24, 2024
28d6bdb
fix: use create permissions when form creates new entity
tomwwinter Jan 24, 2024
587ff54
removed unnecessary variable
TheSlimvReal Jan 25, 2024
017a45b
Merge branch 'master' into tw/feat/1912-filter-fields-if-user-has-no-…
TheSlimvReal Jan 25, 2024
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
Prev Previous commit
Next Next commit
fix: test syntax
Co-authored-by: Simon <simon@aam-digital.com>
  • Loading branch information
tomwwinter and TheSlimvReal authored Jan 16, 2024
commit f99b8d0271d5ad22b932f6ca82c527eed0dd8af6
Original file line number Diff line number Diff line change
@@ -91,9 +91,9 @@ describe("EntityFormService", () => {

service.disableReadOnlyFormControls(formGroup, entity);
TheSlimvReal marked this conversation as resolved.
Show resolved Hide resolved

expect(formGroup.get("name").disabled).toBeTruthy();
expect(formGroup.get("name").disabled).toBeTrue();
expect(formGroup.get("foo").disabled).toBeFalse();
expect(formGroup.get("bar").disabled).toBeTruthy();
expect(formGroup.get("bar").disabled).toBeTrue();
});
TheSlimvReal marked this conversation as resolved.
Show resolved Hide resolved

it("should update entity if saving is successful", async () => {