Skip to content

[CBO-1777] Update googlemaps-services for Ruby 2.7 support#3888

Merged
Joseph Haig (jrmhaig) merged 1 commit intomasterfrom
jh/update_googlemaps-services
Apr 15, 2021
Merged

[CBO-1777] Update googlemaps-services for Ruby 2.7 support#3888
Joseph Haig (jrmhaig) merged 1 commit intomasterfrom
jh/update_googlemaps-services

Conversation

@jrmhaig
Copy link
Contributor

What

Update the googlemaps-services gem to use this branch: https://github.com/jrmhaig/googlemaps-services/tree/update_http

Ticket

Mileage calculation intermittent failures

Why

The googlemaps-services gem has a dependency on the http gem pinned to a version that does not support Ruby 2.7. This results in intermittent failures. A PR has been submitted to googlemaps-services (amrfaissal/googlemaps-services#13) but it has not yet been merged.

Once the PR has been merged and a new version released then we need to update Gemfile to use the released version again.

@jsugarman
Copy link
Contributor

I notice this gem is not very popular. from a quick view i wonder whether this one is worth a look as seesm more popular but even older

https://github.com/edwardsamuel/google-maps-services-ruby

@jrmhaig
Copy link
Contributor Author

Joseph Haig (jrmhaig) commented Apr 15, 2021

Joel Sugarman (@jsugarman) I am currently looking at the geocoder gem to see if we can use it instead.

Update: It looks as though our use case is out of the scope of geocoder so that is not the answer. See alexreisner/geocoder#815

This is using this branch on Github:

  https://github.com/jrmhaig/googlemaps-services/tree/update_http

One it has been merged and a new version released then that should be
used instead (see amrfaissal/googlemaps-services#13)
@jrmhaig Joseph Haig (jrmhaig) force-pushed the jh/update_googlemaps-services branch from 67ea0d9 to 4058844 Compare April 15, 2021 15:21
Copy link
Contributor

@jsugarman Joel Sugarman (jsugarman) left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 as an interim measure this looks good

@jrmhaig Joseph Haig (jrmhaig) merged commit 77c3c79 into master Apr 15, 2021
@jrmhaig Joseph Haig (jrmhaig) deleted the jh/update_googlemaps-services branch April 15, 2021 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants