Skip to content
This repository was archived by the owner on May 7, 2025. It is now read-only.

Conversation

danielduhh
Copy link

Description:

Couple things going on here.

  1. The hosted font files were on a unreliable server db.onlinewebfonts.com, so i replaced it with the static files being used by chbp.com

Screen Shot 2019-07-19 at 5 23 00 PM

  1. Reworked the geolocate callback to parse the location and send it off to GA

UI images:

Please add images for any changes to the UI.

Unit Test Approach:

Sending off the location like it should...

Screen Shot 2019-07-19 at 5 35 21 PM

Test Results:

  • Did you run npm test and did the results pass?
  • Did you add unit tests that cover your changes?

Describe other (non-unit) test results here.

@danielduhh danielduhh requested review from ccabanero and cbessee July 20, 2019 00:41
@danielduhh danielduhh merged commit 305b9dc into spatialdev:development Jul 20, 2019
danielduhh pushed a commit that referenced this pull request Jul 20, 2019
* fix reset app bug (#52)

* updates registerServiceWorker, confirms sw-build and sw-template look good, updates package.json to run customized build (#54)

* Geolocate callback & reliable fonts (#56)

* Send the users location to ga on successful locate
Use a more reliable ttf font file

* geolocate callback

* travis

* weird formatting

* no longer need to trigger

* Revert "updates registerServiceWorker, confirms sw-build and sw-template look good, updates package.json to run customized build (#54)"

This reverts commit 3a1158f.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant