Open jeremy-jung1 opened 1 week ago
Thanks for the contribution! It looks like @jeremy-jung1 is an internal user so signing the CLA is not required. However, we need to confirm this.
For some reason the linter isn't working, i'm surprised there is var
in the code and tests are passing... let's only use let
or const
. No var
in the code base.
When I opened the modal, I could see some errors/warnings in the browser console. Let's fix them, because otherwise that won't leave a good impression to other developers.
W-15628148 and W-15621326
Description
Added "Load More" button functionality to store locator and integrated native web geolocation API to "Use my Location" function. Made default of country selector, if defined in
constants.js
, to show "Select a country". Updated UI text to remove distance and show information on geolocation.Types of Changes
Changes
How to Test-Drive This PR
Load More testing
Use my location testing
Testing the store locator page
Checklists
General
Accessibility Compliance
You must check off all items in one of the follow two lists:
or...
Localization