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-product.smart-*: the zoneId param should be optional #1176

Closed
florian-sanders-cc opened this issue Oct 4, 2024 · 0 comments
Closed
Assignees
Labels
bug Something isn't working easy-quick-win

Comments

@florian-sanders-cc
Copy link
Contributor

florian-sanders-cc commented Oct 4, 2024

Context

Docs for the cc-pricing-product smart components specify that zoneId has a default value ("par") which implies they should be optional.

In reality, zoneId is not declared as optional and doesn't have a default value so we should fix that.

@florian-sanders-cc florian-sanders-cc added bug Something isn't working easy-quick-win labels Oct 4, 2024
@florian-sanders-cc florian-sanders-cc self-assigned this Oct 4, 2024
florian-sanders-cc added a commit that referenced this issue Oct 30, 2024
Smart components:
- The `addonFeatures` context parameter of the `cc-pricing-product.smart-addon`
component is now optional as it should always have been
(default value: `[]` meaning that all features are displayed by default).
- The `zoneId` context parameter of both `cc-pricing-product.smart-addon` &
`cc-pricing-product.smart-runtime` is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

Fixes #1167 #1109 #1174 #1176
florian-sanders-cc added a commit that referenced this issue Oct 30, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167 #1176 #1109
florian-sanders-cc added a commit that referenced this issue Oct 30, 2024
Smart components:
- The `addonFeatures` context parameter of the `cc-pricing-product.smart-addon`
component is now optional as it should always have been
(default value: `[]` meaning that all features are displayed by default).
- The `zoneId` context parameter of both `cc-pricing-product.smart-addon` &
`cc-pricing-product.smart-runtime` is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

Fixes #1167 #1109 #1174 #1176
florian-sanders-cc added a commit that referenced this issue Oct 30, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167 #1176 #1109
florian-sanders-cc added a commit that referenced this issue Nov 20, 2024
Smart components:
- The `addonFeatures` context parameter of the `cc-pricing-product.smart-addon`
component is now optional as it should always have been
(default value: `[]` meaning that all features are displayed by default).
- The `zoneId` context parameter of both `cc-pricing-product.smart-addon` &
`cc-pricing-product.smart-runtime` is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

Fixes #1167
Fixes #1109
Fixes #1174
Fixes #1176
florian-sanders-cc added a commit that referenced this issue Nov 20, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167
Fixes #1176
Fixes #1109
florian-sanders-cc added a commit that referenced this issue Nov 25, 2024
Smart components:
- The `addonFeatures` context parameter of the `cc-pricing-product.smart-addon`
component is now optional as it should always have been
(default value: `[]` meaning that all features are displayed by default).
- The `zoneId` context parameter of both `cc-pricing-product.smart-addon` &
`cc-pricing-product.smart-runtime` is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

Fixes #1167
Fixes #1109
Fixes #1174
Fixes #1176
florian-sanders-cc added a commit that referenced this issue Nov 25, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167
Fixes #1176
Fixes #1109
florian-sanders-cc added a commit that referenced this issue Nov 29, 2024
Smart components:
- The `addonFeatures` context parameter of the `cc-pricing-product.smart-addon`
component is now optional as it should always have been
(default value: `[]` meaning that all features are displayed by default).
- The `zoneId` context parameter of both `cc-pricing-product.smart-addon` &
`cc-pricing-product.smart-runtime` is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

Fixes #1167
Fixes #1109
Fixes #1174
Fixes #1176
florian-sanders-cc added a commit that referenced this issue Nov 29, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167
Fixes #1176
Fixes #1109
florian-sanders-cc added a commit that referenced this issue Dec 4, 2024
Smart components:
- The `addonFeatures` context parameter of the `cc-pricing-product.smart-addon`
component is now optional as it should always have been
(default value: `[]` meaning that all features are displayed by default).
- The `zoneId` context parameter of both `cc-pricing-product.smart-addon` &
`cc-pricing-product.smart-runtime` is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

The payload type of the `cc-pricing-product:add-plan` dispatched event has changed.
Plans must now include a `priceId` property to be handled by the
`cc-pricing-estimation` correctly. This also means that custom products
/ plans can no longer be added to the estimation because they are not
part of the priceSystem and have no `priceId`.

Fixes #1167
Fixes #1109
Fixes #1174
Fixes #1176
florian-sanders-cc added a commit that referenced this issue Dec 4, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167
Fixes #1176
Fixes #1109
florian-sanders-cc added a commit that referenced this issue Dec 5, 2024
…urrency

Smart components:
- The `zoneId` context parameter of `cc-pricing-product-consumption.smart`
is now optional (default value: `'par'`).

BREAKING CHANGE: the component no longer supports hard coded change rate
Prices displayed in pricing components are no longer computed based on
hard coded change rate.
Prices related to the selected currency come from the API priceSytem
endpoint.
The component also relies on the new `state` prop for data coming from
the API.

This means that some types for the component properties have changed:
- `currency` is now a `string` (ISO 4217 currency code),
- `product` has been removed,
- `state` has been added to replace `product`,
  - `product.state` is now `state.type`.

The type of the payload dispatched with the event
`cc-pricing-product:add-plan` has changed:
- `sections` (`Array<PricingSection>`) are now part of the payload so
  that prices can be computed by the `cc-pricing-estimation` component
  everytime the currency changes.

Fixes #1167
Fixes #1176
Fixes #1109
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working easy-quick-win
Projects
None yet
Development

No branches or pull requests

1 participant