Please review:
Google Maps API warning: SensorNotRequired: https://developers.google.com/maps/documentation/javascript/error-messages

The sensor parameter is no longer required for the Google Maps JavaScript API. It won't prevent the Google Maps JavaScript API from working correctly, but we recommend that you remove the sensor parameter from the script element.

I added patch to cleanup module.

CommentFileSizeAuthor
#5 sensornotrequired_warning-2634670-5.patch1.66 KBcebasqueira
geolocation_googlemaps-sensor-not-required.patch1.64 KBgugalamaciek

Comments

ngwebs created an issue. See original summary.

derjochenmeyer’s picture

derjochenmeyer commented
Status: Active » Needs review

Thanks for the patch!

Status: Needs review » Needs work

The last submitted patch, geolocation_googlemaps-sensor-not-required.patch, failed testing.

cebasqueira’s picture

cebasqueira commented
Assigned: Unassigned » cebasqueira
cebasqueira’s picture

cebasqueira commented
Assigned: cebasqueira » Unassigned
Status: Needs work » Needs review
StatusFileSize
new sensornotrequired_warning-2634670-5.patch1.66 KB

Status: Needs review » Needs work

The last submitted patch, 5: sensornotrequired_warning-2634670-5.patch, failed testing.

cebasqueira’s picture

cebasqueira commented
Status: Needs work » Needs review
faline’s picture

faline commented
Status: Needs review » Reviewed & tested by the community

Test the patch on #5 and it is working.
There is no warning for sensor anymore.

christianadamski’s picture

christianadamski commented
Status: Reviewed & tested by the community » Fixed

Inserted your changes without testing them. Pushed. Thank you!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

josueValRob’s picture

josueValRob commented

Do you have something for drupal 7??