Closed florian-sanders-cc closed 3 weeks ago
🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/cc-zone-related-components/state-migration/index.html.
This preview will be deleted once this PR is closed.
🔎 The preview has been automatically deleted.
:warning: Please review this thoroughly: components impacted by this PR are essential parts of the console and some of them are tricky (hey there
cc-header-app
).What does this PR do?
cc-zone
,cc-zone-input
,cc-header-app
,cc-header-addon
components to implement our new state structure,cc-pricing-header
andcc-addon-linked-apps
components to adapt to thecc-zone
new state,How to review?
cc-zone
component? Sometimes it's fairly relevant (withincc-header-app
andcc-header-addon
, it is used on its own and it can be set to loading / loaded independently from the rest of the components) but in other cases (cc-zone-input
,cc-pricing-header
,cc-addon-linked-apps
) there is no way it can be set toloading
. We could go for a simple skeleton prop instead of a state?cc-header-app
&cc-header-addon
are special because they fetch data from 3 different sources and only one source (app / addon info may trigger the error state). This should be ISO with what we currently have in prod (unless I made a mistake which is definitely possible),cc-zone
cc-zone-input
cc-header-app
cc-header-addon
cc-pricing-header
&cc-addon-linked-apps
, it may be easier to check the diffs but it's up to you,cc-zone
- prod vscc-zone
- preview,cc-zone-input
- prod vscc-zone-input
- preview,cc-header-app
- prod vscc-header-app
- preview,cc-header-addon
- prod vscc-header-addon
- preview,cc-pricing-header
- prod vscc-pricing-header
- preview,cc-addon-linked-apps
- prod vscc-addon-linked-apps
- preview,cc-pricing-header
&cc-addon-linked-app
using thedemo-smart
page.