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

Fix infinite loop with layout cards module #132

Merged
merged 3 commits into from
Nov 7, 2023

Conversation

nikomakela
Copy link
Contributor

@nikomakela nikomakela commented Nov 6, 2023

HCRC-106 HCRC-108.

When a layout cards module wsa used, the CMS pages went to an infite rerendering loop in apps.

This can be tested in Liikunta test-CMS-env with /fi/pages/moduulitesti/


Also fixed a style filling bug in Card-module:

Broken:

image

Fixed:
image

Mobile:

image

@nikomakela nikomakela marked this pull request as draft November 6, 2023 10:06
@nikomakela nikomakela force-pushed the HCRC-106-fix-inifinite-loop-on-cards-module branch from c1fa7da to 39b24a6 Compare November 6, 2023 10:14
@nikomakela nikomakela force-pushed the HCRC-106-fix-inifinite-loop-on-cards-module branch 2 times, most recently from 629bf2d to fc5aff3 Compare November 6, 2023 13:58
HCRC-106 HCRC-108
Use a crypto lib to create an unique hash key.
Add a missing key-property to layout cards module, 
steps module and the modules in the pageContent component.
@nikomakela nikomakela force-pushed the HCRC-106-fix-inifinite-loop-on-cards-module branch from fc5aff3 to f6c8721 Compare November 7, 2023 09:12
@nikomakela nikomakela marked this pull request as ready for review November 7, 2023 09:12
@nikomakela nikomakela merged commit 6772d06 into main Nov 7, 2023
1 check passed
@nikomakela nikomakela deleted the HCRC-106-fix-inifinite-loop-on-cards-module branch November 7, 2023 09:14
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.

3 participants