Closed jeremy-jung1 closed 3 months ago
@jeremy-jung1 FYI I've left some comments. But I see that you've just made new code changes. I'll review them later at another time.
Before deleting the translations in this PR for professional translation, I took a screenshot of German locale store locator to show that the store locator is translatable.
Not a part of this ticket, but I would be interested in seeing how we handle saving the shopper's selected store in the future since it might have some implication on hybrid (ie. how do we share the selected store from PWA to SFRA?).
Could you loop me in once we get to that part of the epic?
Description
W-15621253
This is the initial Store Locator implementation, which is without pagination and linking of geolocation web API. The initial implementation includes:
The PR will merge into a store locator base branch, to which subsequent PRs will be merged until the store locator is ready for shipping. Subsequent PRs will link the geolocation web API to the store locator and enable pagination for the store locator.
Types of Changes
Changes
How to Test-Drive This PR
npm run test
Checklists
General
Accessibility Compliance
You must check off all items in one of the follow two lists:
or...
Localization