Completely refactored our React components into as many sub components as possible.
This was done in order to accomplish several outcomes:
Semantically named components.
Smaller components for easier readability and modification.
To allow for component reuse.
One major result was the discovery of some duplicated components accomplishing the same thing. A primary example is the ProductQuantity code used for adding to and removing from the shopping cart. This cart mutability functionality is found throughout the codebase inside of numerous UI components and so it makes a lot more sense to reuse them rather than redefine them.
Code Review Checklist
Description
Completely refactored our React components into as many sub components as possible.
This was done in order to accomplish several outcomes:
One major result was the discovery of some duplicated components accomplishing the same thing. A primary example is the ProductQuantity code used for adding to and removing from the shopping cart. This cart mutability functionality is found throughout the codebase inside of numerous UI components and so it makes a lot more sense to reuse them rather than redefine them.
Type of change
How was it tested?