aligent / magento-stockists-module

GNU General Public License v3.0
1 stars 1 forks source link

Improve country, region, and trading hour changes #9

Closed brettlaishley closed 3 years ago

brettlaishley commented 3 years ago

At the moment the region_id is saved to the region field, instead of the full region name like 'South Australia', this PR allows the DB entry and hence graphql queries to return a nice name instead of a meaningless ID.

Also fixes a bug where the 'State/Province' field shows as a text input all the time instead of when the selected country requires a region to be selected ... It does not auto populate the region on first load of the page, but it does force the user to select a region when they switch the country away from Australia and back to Australia.

brettlaishley commented 3 years ago

If someone could approve the final changes I've brought in that would be great, I've stopped tinkering with the branch now.