Skip to content
This repository has been archived by the owner on Mar 22, 2019. It is now read-only.

Broken Google maps #3749

Open
simonihmig opened this issue Dec 26, 2018 · 7 comments
Open

Broken Google maps #3749

simonihmig opened this issue Dec 26, 2018 · 7 comments
Labels

Comments

@simonihmig
Copy link
Contributor

simonihmig commented Dec 26, 2018

Current Behavior

Google Map cannot be loaded. See below for a screenshot.

Expected Behavior

Google Maps should load correctly.

Steps to Reproduce

  1. visit https://www.emberjs.com/community/meetups/

Possible Solution

Seems GMaps is loaded without an API key, which does not work anymore: https://developers.google.com/maps/documentation/javascript/error-messages?utm_source=maps_js&utm_medium=degraded&utm_campaign=keyless#api-key-and-billing-errors

Sources

image

@kennethlarsen
Copy link
Member

Thanks for posting this @simonihmig!

I think we need to use Leaflet here like it was done in the Super Rentals app: https://github.com/ember-learn/super-rentals/pull/110/files

@mansona
Copy link
Member

mansona commented Dec 28, 2018

Howdy folks 👋

I've just implemented leaflet in the new website here ember-learn/ember-website#96 but I don't know if we need to fix it on the old one before the new one is launched 🤔

@MelSumner any thoughts? do you think we can hold off and just release the new website to fix this?

@MinThaMie
Copy link
Contributor

I think just the API key they recently introduced is missing: https://developers.google.com/maps/documentation/javascript/get-api-key

@simonihmig
Copy link
Contributor Author

I think just the API key they recently introduced is missing

Yes, but that requires providing billing details, and costs could occur depending on the actual usage volume. So that's probably a blocker for an OSS project without a corporate backer willing to pay that!?

@MinThaMie
Copy link
Contributor

True :( It's sad that they choose to do this...

@mansona
Copy link
Member

mansona commented Feb 13, 2019

I wouldn't worry about this issue too much as it is solved on the new website: https://ember-beta-website.netlify.com/community/meetups

I don't feel comfortable closing this issue until the swap-over has actually happened though 🤔

@simonihmig
Copy link
Contributor Author

@mansona nice!

Had to file an issue though: ember-learn/ember-website#152 🤷‍♂️

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

No branches or pull requests

4 participants