elivz / VzAddress-Craft

Address fieldtype for the Craft CMS
30 stars 13 forks source link

Error outputting dynamic Google Map #29

Closed artmem closed 6 years ago

artmem commented 7 years ago

I'm trying to output a dynamic map using this code: {{ contact.address.dynamicMap() }} On my local machine, it works fine, but uploaded to my server, I just get a grey box and these two js errors, referencing maps.googleapis.com: Line 35:336 InvalidValueError: setPosition: not a LatLng or LatLngLiteral: not an Object Line 35:336 InvalidValueError: setCenter: not a LatLng or LatLngLiteral: not an Object

elivz commented 7 years ago

Did you create an API key and enter it in the plugin settings? Without a valid API key, Google won't geocode the address into a set of coordinates, which would explain that error. Your local machine was probably grandfathered in from before last year when they started requiring a key for all API access.

Let me know if that wasn't the problem, and I'll see what else I can come up with. I will be traveling the next few days though, so I might not be able to respond until the weekend.

artmem commented 7 years ago

Now I'm more confused. I did create an API key. I tested it last night from home, and the map worked fine. Now, I'm back at work, and it isn't working on multiple computers. I thought it might be a network issue, but it isn't loading on my phone, either. I have no idea what's going on. The site is dev.crasianbrewing.com

elivz commented 7 years ago

Can you try downloading the current version from GitHub? I think I found the problem, but I want to confirm that it's fixed before I release it officially.

artmem commented 7 years ago

Thanks. I just left for vacation, so I won't be able to test it until the 15th. I'll let you know if it works.

On Aug 4, 2017, at 5:50 PM, Eli Van Zoeren notifications@github.com wrote:

Can you try downloading the current version from GitHub? I think I found the problem, but I want to confirm that it's fixed before I release it officially.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

artmem commented 7 years ago

Just installed the new version. Now it just displays a message that a Google API key is required, even though I have one, and it shows up in the plugin settings.

elivz commented 7 years ago

Nathaniel,

I have not been able to duplicate this with the current version. Is there any change I could get access to your site to try a couple things? If so, you can email me at eli@elivz.com. I'm not sure what is different between your setup and my test sites....

curtishenson commented 6 years ago

Just ran into the "Google API key is required" issue, I think it's an error in the logic for grabbing the key. Submitted a pull request of what I think is the fix.

artmem commented 6 years ago

Sorry, I stopped getting updates from this thread, for some reason. Eli, I just emailed you info tho access my site.

elivz commented 6 years ago

I just merged a pull request from @curtishenson (thanks!) that should fix this for you.