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.

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

Jump to bottom

remove permissions from node/content modify flow #10443

Closed
vbradnitski opened this issue Mar 6, 2024 · 0 comments · Fixed by #10492
Closed

remove permissions from node/content modify flow #10443

vbradnitski opened this issue Mar 6, 2024 · 0 comments · Fixed by #10492
Assignees
@vbradnitski
Labels
Improvement
Milestone
8.0.0

Comments

@vbradnitski
Copy link
Contributor

it should be restricted to changing Node/Content permissions data through 'modify'. Only 'applyPermissions' should be used for that.
@vbradnitski vbradnitski self-assigned this Mar 6, 2024
@rymsha rymsha added this to the 8.0.0 milestone Mar 15, 2024
vbradnitski added a commit that referenced this issue Mar 29, 2024
@vbradnitski
remove permissions from node/content modify flow
fc70674 
#10443
vbradnitski added a commit that referenced this issue Mar 29, 2024
@vbradnitski
remove permissions from node/content modify flow #10443
6c20a6e
vbradnitski added a commit that referenced this issue Mar 29, 2024
@vbradnitski
remove permissions from node/content modify flow #10443
f50b9dd
vbradnitski added a commit that referenced this issue Mar 29, 2024
@vbradnitski
remove permissions from node/content modify flow #10443
38dc284
vbradnitski added a commit that referenced this issue Mar 29, 2024
@vbradnitski
remove permissions from node/content modify flow #10443
4b0f9fd
@vbradnitski vbradnitski linked a pull request Apr 1, 2024 that will close this issue
remove permissions from node/content modify flow #10443 #10492
Merged
@rymsha rymsha mentioned this issue Apr 2, 2024
Closed
vbradnitski added a commit that referenced this issue Apr 2, 2024
@vbradnitski
remove permissions from node/content modify flow #10443
2be2208
vbradnitski added a commit that referenced this issue Apr 2, 2024
@vbradnitski
remove permissions from node/content modify flow #10443 (#10492)
651488c
vbradnitski added a commit that referenced this issue Apr 24, 2024
@vbradnitski
remove permissions from node/content modify flow #10443 (#10492)
b8b298a
rymsha pushed a commit that referenced this issue May 6, 2024
@vbradnitski
Permission patching (#10523)
0896641 
* remove inheritPermissions field #10424

* permissions update for published nodes #10420 (#10428)

* get 'inheritPermissions' flag back for Node and Content #10442

* target branches are upgraded with obsolete version data #10454

* js api changes #10489 (#10493)

* remove permissions from node/content modify flow #10443 (#10492)

* merge with existing permissions #10494 (#10499)

* applyPermissions - subtree selection #10485 (#10515)

* rename mode to scope #10520 (#10521)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vbradnitski vbradnitski

Labels
Improvement
Projects
None yet
Milestone
8.0.0
Development

Successfully merging a pull request may close this issue.

remove permissions from node/content modify flow #10443 enonic/xp
3 participants
@rymsha @vbradnitski @alansemenov