Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

Fix map markers on qiime and emp #640

Merged
merged 4 commits into from Mar 7, 2014
Merged

Conversation

teravest
Copy link
Contributor

@teravest teravest commented Mar 6, 2014

The map markers stopped working about a day ago but still worked on american gut.

I replaced the setMarkers function with the version of the function from the americangut page.

Markers are displaying in firefox, Safari, and Chrome now.

@adamrp
Copy link
Member

adamrp commented Mar 6, 2014

If it works, it works.. I just wish we knew why it stopped working. Do you think we can look into that a little bit before we merge? It just seems weird to me that it would randomly stop working when we have changed nothing in our code... maybe the google maps API changed?

@ElDeveloper
Copy link
Member

👍

@ElDeveloper
Copy link
Member

@adamrp I'm going to bet right now that the problem is Google's API changing, from what @teravest mentioned in one of her e-mails.

@teravest
Copy link
Contributor Author

teravest commented Mar 6, 2014

This is on webdev now

@ElDeveloper
Copy link
Member

Thanks for deploying this @teravest, just noticed that the markers are showing a "person" whilst these should be the generic Google Maps markers, aside from that, it looks great.

@adamrp
Copy link
Member

adamrp commented Mar 6, 2014

Yep, the icons should be changed 👍 Other than that, I'm happy with it! Thanks, @teravest

@teravest
Copy link
Contributor Author

teravest commented Mar 6, 2014

Updated on webdev with no men

ElDeveloper added a commit that referenced this pull request Mar 7, 2014
Fix map markers on qiime and emp
@ElDeveloper ElDeveloper merged commit 5c9e7b0 into qiime:master Mar 7, 2014
@ElDeveloper
Copy link
Member

ginrhpd

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

Successfully merging this pull request may close these issues.

None yet

3 participants