Skip to content

fix: estimate arrival times using google maps#567

Merged
sahil28297 merged 2 commits intoBloomstack:bs-mar-sprint-1from
hrwX:delivery_trip_estimates
Mar 23, 2021
Merged

fix: estimate arrival times using google maps#567
sahil28297 merged 2 commits intoBloomstack:bs-mar-sprint-1from
hrwX:delivery_trip_estimates

Conversation

@hrwX
Copy link
Contributor

@hrwX hrwX commented Mar 19, 2021 •
edited
Loading

  • Since estimated times are not reported to bloomtrace, the METRC Integration breaks
  • Sync only those Delivery Notes which has Delivery Trip associated
  • Using Google Maps Integration if estimated times are absent
  • If Google Maps Integration is disabled, throw an error before syncing with bloomtrace

neilLasrado
neilLasrado previously approved these changes Mar 19, 2021
View reviewed changes
@neilLasrado
Copy link
Contributor

neilLasrado commented Mar 19, 2021

@hrwX Should we let the users to manually set the arrival times incase they don't want a google integration?

@sahil28297 sahil28297 merged commit 59f527c into Bloomstack:bs-mar-sprint-1 Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@neilLasrado neilLasrado neilLasrado approved these changes

@sahil28297 sahil28297 sahil28297 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@hrwX @neilLasrado @sahil28297