Xevib / osmbot

OpenStreetMap Telegram bot
GNU General Public License v3.0
43 stars 12 forks source link

Add support to tel URI for phone and mobile data #263

Open airon90 opened 3 years ago

airon90 commented 3 years ago

Right now, a telephone number is shown in plain text. The bot should use link in order to start a call: instead of +XXXXXXX there should be an URL to tel:+XXXXXXX

Xevib commented 3 years ago

Thank you Michael I will change it as soon I can

airon90 commented 3 years ago

Thank you :)

Xevib commented 3 years ago

@airon90 I checked it and I have two doubts. In wich Telegram application it happens? (Telegram desktop ,mobile Android or IOs) In wich command happens? on /detailsnod1819099866 or /phonenod1819099866

airon90 commented 3 years ago

In wich Telegram application it happens? (Telegram desktop ,mobile Android or IOs)

Mobile (Android) and desktop

In wich command happens? on /detailsnod1819099866 or /phonenod1819099866

No command, I used the inline bot: I wrote @osmbot, looked for the element I was looking for and chose it. I tested /detailsnod5976037776 right now and it shows raw data of phone too. /phonenod5976037776 is not working.

FYI the element is https://www.openstreetmap.org/node/5976037776

Xevib commented 3 years ago

@airon90 I did the change but it seems that this depends on the Telegram client https://github.com/Xevib/osmbot/commit/2a9d474c6e597311b42f41eaebe4fd244e30ce1c

Seems there is more people with the same issue https://stackoverflow.com/questions/40590683/how-to-make-phone-number-a-link-in-telegram-bot