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

Refactor Covers block to use hydration #1104

Merged
merged 1 commit into from Oct 17, 2023
Merged

Conversation

mleray
Copy link
Contributor

@mleray mleray commented Aug 22, 2023

Description

See PLANET-6917
This is to stop using the deprecated frontendRendered function

Testing

You can make sure that the Covers block still works as expected, either on local or on the oberon test instance

@mleray mleray self-assigned this Aug 22, 2023
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Aug 22, 2023
/unhold f959dcbc-18cd-4568-be02-54d5429a522f
@planet-4
Copy link
Contributor

planet-4 commented Aug 22, 2023

Test instance is ready 🚀

🌑 oberon | admin | blocks report | CircleCI | composer-local.json

⌚ 2023.09.25 08:44:46

@mleray mleray force-pushed the covers-block-hydration-refactor branch from e30d408 to 0d638e9 Compare August 22, 2023 13:25
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Aug 22, 2023
/unhold c7ab61f3-ec2f-41ab-84de-50c680d4a0e9
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Aug 24, 2023
/unhold e84b88d4-f856-4d79-852d-a15ba1654898
@mleray mleray force-pushed the covers-block-hydration-refactor branch 2 times, most recently from 0d89614 to 8929c03 Compare August 25, 2023 12:15
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Aug 25, 2023
/unhold 967f61dd-4214-4da9-8f69-038ce4b2dfcd
@mleray mleray marked this pull request as ready for review August 25, 2023 12:43
@mleray mleray added the Review label Aug 25, 2023
@mleray mleray force-pushed the covers-block-hydration-refactor branch from 8929c03 to 45b7f6e Compare August 31, 2023 12:34
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Aug 31, 2023
/unhold 5075f23f-cc75-4113-981a-8e033ae1e431
@mleray mleray force-pushed the covers-block-hydration-refactor branch from 45b7f6e to a30c1f5 Compare September 12, 2023 07:05
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Sep 12, 2023
/unhold 6bfc7518-326d-4b0b-947c-67f7bd2963c3
@mleray mleray force-pushed the covers-block-hydration-refactor branch from a30c1f5 to 210f1e7 Compare September 12, 2023 07:12
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Sep 12, 2023
/unhold 9418b41e-c3e9-4b3b-a72d-b15fe248fe93
@mleray mleray force-pushed the covers-block-hydration-refactor branch from 210f1e7 to 28bcbde Compare September 12, 2023 07:34
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Sep 12, 2023
/unhold 12682693-3588-4711-918f-fca527ff7b1a
@mleray mleray force-pushed the covers-block-hydration-refactor branch from 28bcbde to 864a366 Compare September 15, 2023 10:04
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Sep 15, 2023
/unhold 96185443-fd13-49ef-b7f9-be80c40010d5
@mleray mleray force-pushed the covers-block-hydration-refactor branch from 864a366 to 3abbc83 Compare September 18, 2023 08:05
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Sep 18, 2023
/unhold 5e45354a-5a31-47df-84be-acaeb139d415
This is to stop using the deprecated frontendRendered function
@mleray mleray force-pushed the covers-block-hydration-refactor branch from 3abbc83 to 68a0e75 Compare September 25, 2023 08:37
planet-4 added a commit to greenpeace/planet4-test-oberon that referenced this pull request Sep 25, 2023
/unhold 12c71225-f41f-42e2-967b-14be35da230d
@mleray
Copy link
Contributor Author

mleray commented Sep 25, 2023

SonarCloud complains about duplicated code, it's actually not related to these changes, but I think in this case it makes sense to keep it anyway 🤔

Copy link
Member

@lithrel lithrel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lithrel lithrel merged commit 6cbdd62 into main Oct 17, 2023
13 of 14 checks passed
@lithrel lithrel deleted the covers-block-hydration-refactor branch October 17, 2023 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants