openaq / openaq-metadata-editor

MIT License
1 stars 1 forks source link

Error on page load #73

Closed jflasher closed 4 years ago

jflasher commented 4 years ago

https://metadata.openaq.org/ is deployed but seeing 'Error: domain option is required' on load. I think this is likely due to an autho0 var missing? Are we passing this through the build somehow? If you can point me to whatever we need to add here and how we want to pass it, I should be able to add it. Thanks!

cc/ @olafveerman @sethvincent @necoline

sethvincent commented 4 years ago

The config for production is here and yeah it looks like it needs to have some values filled in. I just made a pr for this: https://github.com/openaq/openaq-metadata-editor/pull/74

I think these steps will also need to be completed to finish the google integration for production: https://auth0.com/docs/connections/social/google

The main thing we need to finish google integration is a client id and client secret from api credentials from the openaq google account.

jflasher commented 4 years ago

Ok, I updated the apiURL with the prod URL and put in the Mapbox info as well. Two outstanding things (as far as I can tell).

sethvincent commented 4 years ago

that map style is in the devseed account so i made a pr to change the token: https://github.com/openaq/openaq-metadata-editor/pull/78

the only thing about the google auth that doesn't work is that it doesn't retain the session across reloads of the page. as far as i know the setup instructions here are the way to make that work https://auth0.com/docs/connections/social/google

jflasher commented 4 years ago

Ok, created new GCP account tied to dev@ and hooked that up, looks to be working correctly. Thanks!