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

Meed Layout Editor - Meeds-io/MIPs#120 #876

Merged
merged 14 commits into from Apr 26, 2024
Merged

Meed Layout Editor - Meeds-io/MIPs#120 #876

merged 14 commits into from Apr 26, 2024

Conversation

boubaker
Copy link
Member

@boubaker boubaker commented Apr 25, 2024

Prior to this change, when a node page references itself as root node, a
StackOverflow Error is observed while copying node.
Prior to this change, after instanciating a , when getting data, an NPE
can be thrown. This change will avoid NPE when getting basic data from a
DTO.
…AFT Nav visibility - MEED-3411 - Meeds-io/MIPs#120 (#846)

Prior to this change, when requesting multiple pages at the same time
from portal, the same page is displayed in all requested pages. This
change will move the selection of current site and page into a
contextual object `PortalRequestContext` that is built for each HTTP
Request, instead to add it in a statefull mode inside the same `WebUI`
Component which is shared between all pages. In addition, this change
will allow to request a single application inside a page to be displayed
in a standalone way (whithout having to use the
`StandaloneAppRequestHandler` and `UIStandaloneApplication` which
duplicates the code and makes it unmaintainable and not tested
continuously).
…#864)

Prior to this change, when requesting a portlet using direct access URL
to retrieve a resource, the Portlet instance isn't found due to the
usage of a randomly generated identifier for Portlet regenerated on each
instanciation. This change will ensure the use the storage identifier
for a WebUI portlet to ensure coherence of Request queries and expected
responses.
…-4975 - Meeds-io/MIPs#120 (#865)

This change will move the mapping of `cssClass` and `borderColor` to be
used on application as well as on containers.
…s#120 (#869)

Prior to this change, when creating new portal and it's not cached in
WebUI, this may lead to a 404 page. This change will ensure to force the
refresh of WebUI components when the UIPortal wasn't cached.
…D-3875 - Meeds-io/MIPs#120 (#872)

This change will allow to:
- Retrieve portlet with a designated mode using `maximizedPortletMode`
Request Parameter
- Return default skin when no selected skin in currently displayed site
- Avoid reloading the whole page when using WebUI ajax query
- include supported portlet modes in Application Registry DTO
…ds-io/MIPs#120 (#873)

This change will allow to define pages layout using sections, cells and
columns.
At the same time a new XSD has been defined with a cleaned useless
entities.
This change will move the 'hidden' class to parent element of portlet
instead of the child element to ensure to not have grid gap applied when
the application is added in a dynamic of fixed section.
@boubaker boubaker marked this pull request as draft April 25, 2024 13:58
Prior to this change, when the 'targetUserNode = targetUserNode.getParent()' returns null, the targetUserNode is getting null, thus an NPE can be thrown and the site get inaccessible. This change ensures to avoid such an exception and returns the original nodePath.
Copy link

sonarcloud bot commented Apr 25, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
14.1% Coverage on New Code (required ≥ 60%)
1 New Critical Issues (required ≤ 0)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@boubaker boubaker marked this pull request as ready for review April 25, 2024 18:04
@boubaker boubaker requested a review from AzmiTouil April 26, 2024 12:36
@boubaker boubaker merged commit 15eb1c3 into develop Apr 26, 2024
4 of 5 checks passed
@boubaker boubaker deleted the meed-merge-mip-120 branch April 26, 2024 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants