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

cc-pricing-*: update components for the new website (clean commit messages) #783

Merged
merged 11 commits into from
Jun 21, 2023

Conversation

florian-sanders-cc
Copy link
Contributor

Fixes #692

Context

These modifications had already been merged (see #716) but the generated changelog from release please was empty because our commit messages did not follow the proper format.

This PR fixes the commit messages format in order to generate a proper changelog.

@florian-sanders-cc florian-sanders-cc self-assigned this Jun 20, 2023
@github-actions
Copy link
Contributor

🔎 A preview has been automatically published : https://clever-components-preview.cellar-c2.services.clever-cloud.com/cc-pricing/update/index.html.

This preview will be deleted once this PR is closed.

@florian-sanders-cc florian-sanders-cc force-pushed the cc-pricing/update branch 4 times, most recently from 5d94771 to 269135c Compare June 20, 2023 18:45
BREAKING CHANGE: see details below
- add new way of handling the component state and passing data to the component.
- remove the heading, desc and image/logo slots.
- use the new smart API.
- directly render a table within the component instead of relying on a
 `cc-pricing-table` sub-component to do so.
- merge the `cc-pricing-table` component and stories into `cc-pricing-product`.
BREAKING CHANGE: The `cc-pricing-table` component
has been merged into the `cc-pricing-product` component.
BREAKING CHANGE: see more details below
- add new way of handling the component state and passing data to the component.
- the smart component now uses the new smart API.
- remove the slots for heading, description and image.
Copy link
Contributor

@pdesoyres-cc pdesoyres-cc left a comment

Choose a reason for hiding this comment

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

LGTM. You can merge

BREAKING CHANGE: see more details below
- add new way of handling the component state and passing data to the component.
- use the new smart API.
- add a temporality selector component.
BREAKING CHANGE: see more details below
- add new way of handling the component state and passing data to the component.
- use the new smart API.
BREAKING CHANGE: We needed to be able to style and move the pricing
components inside the `cc-pricing-page` component a lot more.
For instance, the `cc-pricing-header` can now be positioned wherever one
wants within the `cc-pricing-page` because everything is slotted within
the `cc-pricing-page`.
The smart `cc-pricing-page` smart component was mainly interacting
with `cc-pricing-header` data (fetching `zones`).
This is why we removed the `cc-pricing-page.smart` part and added a
`cc-pricing-header.smart`.
@florian-sanders-cc florian-sanders-cc marked this pull request as ready for review June 21, 2023 08:59
@florian-sanders-cc florian-sanders-cc merged commit ecc03c0 into master Jun 21, 2023
@florian-sanders-cc florian-sanders-cc deleted the cc-pricing/update branch June 21, 2023 08:59
@github-actions
Copy link
Contributor

🔎 The preview has been automatically deleted.

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.

cc-pricing-*: update components to match the new website styles
2 participants