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: Give access to layout editor page to everyone - MEED-7657 - Meeds-io/meeds#2502 #2527

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

boubaker
Copy link
Member

Prior to this change, when adding a page template, the images attached to it are not viewable by External users. This is due to the fact that the access permission of attached image is computed switch page being edited, or if of type page template to current page (layout-editor). This change will fix two bugs:

  • Give access to layout-editor page to anyone to let the ACL made by page permissions instead (by the Space Layout permissions if the page is added in a space)
  • Make the attached image to a fictive page (not really instanciated yet), accessible to all users since the page can be used in any site/space.

Prior to this change, when adding a page template, the images attached to it are not viewable by External users. This is due to the fact that the access permission of attached image is computed switch page being edited, or if of type page template to current page (layout-editor). This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by page permissions instead (by the Space Layout permissions if the page is added in a space)
- Make the attached image to a fictive page (not really instanciated yet), accessible to all users since the page can be used in any site/space.
@boubaker boubaker merged commit 37e637d into feature/whitepaper Oct 24, 2024
2 of 3 checks passed
@boubaker boubaker deleted the meed-7657 branch October 24, 2024 15:04
Copy link

sonarcloud bot commented Oct 24, 2024

exo-swf pushed a commit that referenced this pull request Oct 25, 2024
… (#2527)

Prior to this change, when adding a page template, the images attached
to it are not viewable by External users. This is due to the fact that
the access permission of attached image is computed switch page being
edited, or if of type page template to current page (layout-editor).
This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by
page permissions instead (by the Space Layout permissions if the page is
added in a space)
- Make the attached image to a fictive page (not really instanciated
yet), accessible to all users since the page can be used in any
site/space.
boubaker added a commit that referenced this pull request Oct 25, 2024
… (#2527)

Prior to this change, when adding a page template, the images attached
to it are not viewable by External users. This is due to the fact that
the access permission of attached image is computed switch page being
edited, or if of type page template to current page (layout-editor).
This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by
page permissions instead (by the Space Layout permissions if the page is
added in a space)
- Make the attached image to a fictive page (not really instanciated
yet), accessible to all users since the page can be used in any
site/space.
boubaker added a commit that referenced this pull request Oct 25, 2024
… (#2527)

Prior to this change, when adding a page template, the images attached
to it are not viewable by External users. This is due to the fact that
the access permission of attached image is computed switch page being
edited, or if of type page template to current page (layout-editor).
This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by
page permissions instead (by the Space Layout permissions if the page is
added in a space)
- Make the attached image to a fictive page (not really instanciated
yet), accessible to all users since the page can be used in any
site/space.
boubaker added a commit that referenced this pull request Oct 25, 2024
… (#2527)

Prior to this change, when adding a page template, the images attached
to it are not viewable by External users. This is due to the fact that
the access permission of attached image is computed switch page being
edited, or if of type page template to current page (layout-editor).
This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by
page permissions instead (by the Space Layout permissions if the page is
added in a space)
- Make the attached image to a fictive page (not really instanciated
yet), accessible to all users since the page can be used in any
site/space.
boubaker added a commit that referenced this pull request Oct 29, 2024
… (#2527)

Prior to this change, when adding a page template, the images attached
to it are not viewable by External users. This is due to the fact that
the access permission of attached image is computed switch page being
edited, or if of type page template to current page (layout-editor).
This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by
page permissions instead (by the Space Layout permissions if the page is
added in a space)
- Make the attached image to a fictive page (not really instanciated
yet), accessible to all users since the page can be used in any
site/space.
exo-swf pushed a commit that referenced this pull request Nov 4, 2024
… (#2527)

Prior to this change, when adding a page template, the images attached
to it are not viewable by External users. This is due to the fact that
the access permission of attached image is computed switch page being
edited, or if of type page template to current page (layout-editor).
This change will fix two bugs:
- Give access to layout-editor page to anyone to let the ACL made by
page permissions instead (by the Space Layout permissions if the page is
added in a space)
- Make the attached image to a fictive page (not really instanciated
yet), accessible to all users since the page can be used in any
site/space.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant