Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(delivery-dispatch/routing): introduce lat/lng accuracy #164

Merged
merged 3 commits into from
Nov 22, 2022
Merged

Conversation

sperka
Copy link
Contributor

@sperka sperka commented Nov 22, 2022

Issue #, if available:

Description of changes:

  • by default use 5-decimal rounding on lat/longs (1.1m)
  • add routing retry mechanism if there's a routing error and try on lower accuracy
  • dev scripts require a .env file with profile/region/namespace vars set

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

* by default use 5-digit rounding on lat/longs
* add routing retry mechanism if there's a routing error and try on lower accuracy
@sperka sperka requested a review from mirgj November 22, 2022 13:05
@mirgj mirgj merged commit 4971936 into main Nov 22, 2022
@mirgj mirgj deleted the fix/routing branch November 22, 2022 14:17
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.

None yet

2 participants