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

Merge Space Public Site - Meeds-io/MIPs#151 #981

Merged
merged 6 commits into from
Oct 7, 2024
Merged

Conversation

boubaker
Copy link
Member

@boubaker boubaker commented Oct 2, 2024

@boubaker boubaker changed the title Merge Space Pblic Site - Meeds-io/MIPs#151 Merge Space Public Site - Meeds-io/MIPs#151 Oct 2, 2024
boubaker and others added 6 commits October 7, 2024 11:57
…EED-7555 - Meeds-io/MIPs#151 (#980)

Prior to this change, the security context of currently authenticated
user was accessed from Service Layer while it should be provided by
Presentation Layer. This change will refactor UserACL implementation to
ensure to get the User ACL Identity as parameter in called methods
rather than implicitly getting it from ConversationState.
Meeds-io/MIPs#151

This change will allow to designate a site to not as a default site that should redirected to when the user goes to /.
…io/meeds#2428 (#983)

Prior to this change, the old height attribute defined in portlets was
applied on the whole block of the portlet. This change will cancel this
style application for portlets retrieved using the new layout management
system so that it still be applied on old pages designed with old system
but not applied on pages applied using the new layout.
Copy link

sonarcloud bot commented Oct 7, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
39.9% Coverage on New Code (required ≥ 60%)

See analysis details on SonarCloud

@boubaker boubaker merged commit 1bf377d into develop Oct 7, 2024
3 of 4 checks passed
@boubaker boubaker deleted the merge-mip-151 branch October 7, 2024 12:02
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.

Space Public Site
2 participants