magento / architecture

A place where Magento architectural discussions happen
275 stars 154 forks source link

Added missing place order address fields, added new StoreConfig fields #463

Open paales opened 4 years ago

paales commented 4 years ago

Problem

Solution

~ = deprecated + = added

Requested Reviewers

@DrewML tagged you because you are mentioned in the Component Assignments for the checkout frontend. @paliarush tagged you because you are mentioned in the Component Assignments for the checkout

Closing notes:

What do you guys think? I'm open to discussions, but I think this cleans stuff up a lot. There are a few things I find difficult to oversee:

fooman commented 3 years ago

If we are making changes to the customer inputs I strongly suggest dropping a lot of the required fields and making them optional. The current GraphQL API is very opinionated and introduces requirements which Magento core may not need (OR should not need).

A few fields which should not be required:

Please also get the input from the PWA Studio people to make sure that changes to region and countries mean they are still easily cacheable by apollo (in other words we don't end up needing v3).