Added the possibility to choose between maps, bing, both or none

ryanmitchell / ti-ext-kitchendisplay

Other

14 stars 9 forks source link

Added the possibility to choose between maps, bing, both or none #34

Closed BreakSecurity closed 3 years ago

BreakSecurity commented 3 years ago

More flexibility added

Bing is not working for me @roydeanjr maybe you can look the way it generate the link.

BreakSecurity commented 3 years ago

Maybe I can move bing and maps links to two variables in summary php to avoid repetition, let me know if for you is good like this

roydeanjr commented 3 years ago

For me, it is probably good. I do not have delivery orders so my add of delivery and map links was thinking others would have delivery orders and the convenience of being able to just click the link while looking at the order would be kinda nice. If it works for you, it is good for me; at least for my current use cases.

Thank you for asking......

ryanmitchell commented 3 years ago

Pushed a few changes - if your happy let me know and I'll merge

BreakSecurity commented 3 years ago

You can marge ;)

BreakSecurity commented 3 years ago

Sorry I just noticed that bing only have destination setted and not starting point. I was fooled by destination

ryanmitchell commented 3 years ago

You can get that form the current device location, dont think its needed.