Skip to content

Add travel time attribution/coordinates#24956

Merged
MartinHjelmare merged 5 commits into
home-assistant:devfrom
ljmerza:patch-1
Jul 16, 2019
Merged

Add travel time attribution/coordinates#24956
MartinHjelmare merged 5 commits into
home-assistant:devfrom
ljmerza:patch-1

Conversation

@ljmerza
Copy link
Copy Markdown
Contributor

@ljmerza ljmerza commented Jul 5, 2019

Please allow me to add the attributions and origin/destinations as this will make it far easier for my travel time lovelace card.

@ghost
Copy link
Copy Markdown

ghost commented Jul 5, 2019

Hey there @robbiet480, mind taking a look at this pull request as its been labeled with a integration (google_travel_time) you are listed as a codeowner for? Thanks!

This is a automatic comment generated by codeowners-mention to help ensure issues and pull requests are seen by the right people.

@ljmerza ljmerza changed the title add google travel time attribution add google travel time attribution/coordinates Jul 5, 2019
@ljmerza ljmerza changed the title add google travel time attribution/coordinates add travel time attribution/coordinates Jul 5, 2019
Comment thread homeassistant/components/google_travel_time/sensor.py
@MartinHjelmare MartinHjelmare changed the title add travel time attribution/coordinates Add travel time attribution/coordinates Jul 7, 2019
Copy link
Copy Markdown
Member

@MartinHjelmare MartinHjelmare left a comment

Choose a reason for hiding this comment

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

Thanks!

@MartinHjelmare
Copy link
Copy Markdown
Member

Can be merged when build passes. An admin may need to override the blocked CI check.

@MartinHjelmare
Copy link
Copy Markdown
Member

MartinHjelmare commented Jul 14, 2019

You can try to rebase on latest dev branch and see if that makes the build pass.

@MartinHjelmare MartinHjelmare merged commit 64b9102 into home-assistant:dev Jul 16, 2019
@lock lock Bot locked and limited conversation to collaborators Jul 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants