Bug reported in #help-vaccinate-caller-app. It is possible for a site to be marked as accepting walkins or appointments now, but vaccinateCA does not handle this case.
To address, i changed appointmentsRequired code to appointmentsOffered, and appropriately handle the case where appointments OR walkins are accepted:
This site now says "Appointment information" with the appointment scheduling instructions for a site that accepts walkins and appointments.
(Note: Use your best judgement for time management. If this PR is a minor content adjustment, you might not invest in the full list of QA checks below. But for medium-large PRs, we recommend a thorough review.)
Open the deploy preview and manually perform the following tests with the browser's developer console open. Fix or log any unexpected errors you see in the console. Check off the following manual tests as you go through them. Make sure to resize the screen and check for poorly positioned or spaced items at mobile and tablet sizes (60%+ of our audience is on mobile devices).
Homepage
[x] Click all the links, internal and external, make sure all open the expected content.
[x] Search box lets you search by zip or county
[x] Counties autocomplete and take you to county page
/near-me
[x] Verify searching by geolocation
[x] Verify searching by zip code (Use one you're familiar with and double-check the listings are what you'd expect)
[x] Map should move when you geolocate or search via zip
[x] Run searches using different options of the filters drop down
[x] Ensure map populates with sites that match filter
[x] Vaccination Site Cards show relevant information
[x] Address in Vaccination Site Cards links to Google Maps view
[x] Panning map changes sites listed
County Vaccination Site List page
[x] List of locations displays successfully on load, split into sites with vaccine and without vaccine
[x] Vaccination Site Cards show relevant information
[x] County policy card shows up near top of page
County Policies page
[x] Shows list of policies per county
[x] Has search bar that autocompletes and filters content
Other pages
[x] 'Providers' shows list of providers
[x] 'About Us' shows prose content and FAQ
[x] 'About Us' shows randomized list of coordinators' names
Did you remember to:
[x] Check for errors in the developer console?
[x] Resize the window to check for display/positioning issues at mobile and tablet sizes?
[x] Check if page titles (what shows in the browser tab) are set properly?
Bug reported in #help-vaccinate-caller-app. It is possible for a site to be marked as accepting walkins or appointments now, but vaccinateCA does not handle this case.
To address, i changed
appointmentsRequiredcode toappointmentsOffered, and appropriately handle the case where appointments OR walkins are accepted:This site now says "Appointment information" with the appointment scheduling instructions for a site that accepts walkins and appointments.
Link to Deploy Preview: https://deploy-preview-913--vaccinateca.netlify.app/
Manual Testing (QA)
(Note: Use your best judgement for time management. If this PR is a minor content adjustment, you might not invest in the full list of QA checks below. But for medium-large PRs, we recommend a thorough review.)
Open the deploy preview and manually perform the following tests with the browser's developer console open. Fix or log any unexpected errors you see in the console. Check off the following manual tests as you go through them. Make sure to resize the screen and check for poorly positioned or spaced items at mobile and tablet sizes (60%+ of our audience is on mobile devices).
Homepage
/near-me
County Vaccination Site List page
County Policies page
Other pages
Did you remember to: