urbanriskmap / cognicity-reports-telegram-server

Telegram support for CogniCity GRASP
GNU General Public License v3.0
0 stars 2 forks source link

Location card will not load inside Telegram "browser" on iPhone #1

Closed prittandrani closed 7 years ago

prittandrani commented 7 years ago

Hi @talltom @matthewberryman ! We are in the office and tried to report through telegram by sending messages to @BencanaBot, so cool! There were some issues about the location card:

cc: @anexact

Thank you!

tomasholderness commented 7 years ago

Notes:

tomasholderness commented 7 years ago

Thanks for the information @prittandrani. Just to confirm, this is a bug only with iPhone?

prittandrani commented 7 years ago

@talltom, yes, I tried with another Android phone and it loaded, but not with another iPhone 6.

tomasholderness commented 7 years ago

Hi @prittandrani we've pushed some updates to the cards recently, I wonder when you get chance if you can test this again.

cc. @MaanasaPriyaa @abrahamq as they are working on Telegram support for Chennai

fadmastuti commented 7 years ago

Hi @talltom , I did some tests of the (telegram) cards recently using android. When I just hit the link, this error message appeared:

1

When I did retry the same think, the cards looked just fine. However, I didn't turn my geolocation on. I wonder if it tracked me from my latest location (p.s I am at Depok while it plot me at central Jakarta). But I think it should've warn me to turn on my geolocation before it plots me to the certain location (?)

I also tried to continue reporting with geolocation off. And this error message appeared: 2

cc @prittandrani

tomasholderness commented 7 years ago

Hi Dika,

It looks like the server thinks you already submitted that card (perhaps part of a previous error).

If you ask the bot for a new card do you still get the same error?

Thanks.

On 12 Jan 2017 11:30 a.m., "dika" notifications@github.com wrote:

Hi @talltom https://github.com/talltom , I did some test of the (telegram) cards recently using android. When I just hit the link, this error message appeared:

[image: 1] https://cloud.githubusercontent.com/assets/2441950/21897974/d23748ec-d91d-11e6-80d7-fa37706f46fd.jpeg

When I did retry the same think, the cards looked just fine. However, I didn't turn my geolocation on. I wonder if it tracked me from my latest location (p.s I am at Depok while it plot me at central Jakarta). But I think it should've warn me to turn on my geolocation before it plots me to the certain location (?)

I also tried to continue reporting with geolocation off. And this error message appeared: [image: 2] https://cloud.githubusercontent.com/assets/2441950/21898194/94e56a9a-d91e-11e6-8599-e6ae86ef744f.jpeg

cc @prittandrani https://github.com/prittandrani

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/urbanriskmap/cognicity-reports-telegram/issues/1#issuecomment-272210816, or mute the thread https://github.com/notifications/unsubscribe-auth/AA5qAuU76SOtsDQHeIJehp7yaYQDCFTZks5rRlUPgaJpZM4LfPd_ .

fadmastuti commented 7 years ago

Hi,

Yes. I still get the error message. I refresh the page and try to turn on my geolocation as well but it still appears.

tomasholderness commented 7 years ago

Can you try with this link? https://dev.petabencana.id/cards/Bk8zMVHLe

tomasholderness commented 7 years ago

I haven't been able to recreate the issue yet. I just succesfuly submitted a report from Depok. https://dev.petabencana.id/map/jakarta/478641

fadmastuti commented 7 years ago

success! all good now. I tried to re-type /flood on telegram so I get the updated link I guess and try to make a report again. My bad.. sorry for that :)

tomasholderness commented 7 years ago

No problem. I'm not sure what happened to your previous report - but the key thing is that you can only submit a report once. After that you have to ask the bot for a new card link. We can keep an eye on this and let me know if you experience the same issue again.

fadmastuti commented 7 years ago

will let you know definitely.

btw, should I open a new issue regarding the location? in my opinion - although the map is successfully loaded, not all of the people know where are they on the map if we are not automatically show them their current location. what do you think?

tomasholderness commented 7 years ago

Hi @fadmastuti yes if you could open a new issue that would be great. Thanks

prittandrani commented 7 years ago

@talltom sorry for the late reply, I tried it but still wont work in Telegram and I cant choose "next" button now (last time I can do it without the map loaded) but works well in Safari browser.

From telegram

From Safari

On Fri, Jan 13, 2017 at 00:05 Tomas Holderness notifications@github.com wrote:

Hi @fadmastuti https://github.com/fadmastuti yes if you could open a new issue that would be great. Thanks

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/urbanriskmap/cognicity-reports-telegram/issues/1#issuecomment-272221143, or mute the thread https://github.com/notifications/unsubscribe-auth/AVTdqBRScCieokKWkrG-KH6E08IHqvX9ks5rRl1VgaJpZM4LfPd_ .

matthewberryman commented 7 years ago

Fixed (and also for in-app browsing on Twitter clients on iOS, and Firefox desktop) thanks to some great work by @ojha-url