GoogleWebComponents / google-map

Google Maps web components
https://elements.polymer-project.org/elements/google-map
Other
437 stars 256 forks source link

fix google-map _updateMarkers to set markers property correctly #422

Closed brandonmull closed 6 years ago

brandonmull commented 6 years ago

This pull request probably fixes a number of issues related to markers. The _updateMarkers method in google-map is not setting the markers property correctly. I believe the following commit is to blame: be0881eba858f64bf87b8fe8a2770e86da07fb87. In addition, your documentation needs to be updated to specify that markers need to have attribute slot="markers" in order to be added to the markers property.

googlebot commented 6 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.


What to do if you already signed the CLA

Individual signers
Corporate signers
brandonmull commented 6 years ago

I signed it!

googlebot commented 6 years ago

CLAs look good, thanks!