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: Configure Builtin Sites to be displayed in Sidebar By default - MEED-8323 - Meeds-io/MIPs#175 #160

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

boubaker
Copy link
Member

@boubaker boubaker commented Feb 4, 2025

This change will add an explicit definition of 'displayed' attribute in Default portal definitions in prder to import it in Sidebar by default. The default value of this flag has been changed to false in order to avoid errors while mapping DTO from database or when using the API to create a new Site.

In addition, this change will prepare the default sites layout to use the new Site Layout Definition.

…EED-8323 - Meeds-io/MIPs#175 (#159)

This change will add an explicit definition of 'displayed' attribute in Default portal definitions in prder to import it in Sidebar by default. The default value of this flag has been changed to false in order to avoid errors while mapping DTO from database or when using the API to create a new Site.

In addition, this change will prepare the default sites layout to use the new Site Layout Definition.
@github-actions github-actions bot added the partialCIBuild Perform Partial CI Build label Feb 4, 2025
Copy link

sonarqubecloud bot commented Feb 4, 2025

@boubaker boubaker enabled auto-merge (squash) February 6, 2025 13:30
@boubaker boubaker requested a review from rdenarie February 6, 2025 14:44
@boubaker boubaker merged commit 51b4a8a into develop Feb 6, 2025
7 checks passed
@boubaker boubaker deleted the merge-mip-175 branch February 6, 2025 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
partialCIBuild Perform Partial CI Build
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants