flagship
flagship copied to clipboard
Summary is redundant in ReviewDetails
In ReviewDetails: https://github.com/brandingbrand/flagship/blob/b6ba811acea1db8b5b2da201fcbc3b45f6062106/packages/fscommerce/src/Review/types/ReviewDetails.ts summary is completely redundant to statistics, because statistics extends summary and all of its new keys are optional. This can cause confusion because a user won't know whether to put something into summary or statistics.
Another redundant value: https://github.com/brandingbrand/flagship/blob/b6ba811acea1db8b5b2da201fcbc3b45f6062106/packages/fscomponents/src/components/ProductItem/ProductItem.tsx#L50
swatchesProps also contains items: SwatchItemType[]
These are merged together in ProductItemSwatches, though the component only checks to see if swatchItems has a list, not if swatchesProps.items does: https://github.com/brandingbrand/flagship/blob/b6ba811acea1db8b5b2da201fcbc3b45f6062106/packages/fscomponents/src/components/ProductItem/components/ProductItemSwatches.tsx#L35