search

ubports / telegram-app

Unofficial Telegram App for Ubuntu Desktop & Mobile
GNU General Public License v3.0
55 stars 23 forks source link

Cannot delete history / leave chats with people who have deleted their account #42

Closed bqphone closed 6 years ago

bqphone commented 7 years ago

Hi there,

I am using bq phone E4.5 OS ubuntu 15.04 (r6) (just flashed my phone to ubports latest version). Telegram for ubuntu v2.4.40(0)

I am failing to delete or leave chats with people who deleted (uninstalled) their telegram account. It works fine with people that are currently using telegram.

Flohack74 commented 7 years ago

Confirmed!

dfiloni commented 7 years ago

Could you please test issue-42 branch? The patch is rough but maybe it fixes the issue. Thanks.

bqphone commented 7 years ago

I didn't get any update notification for telegram yet. Till then the problem still exists.

Flohack74 commented 7 years ago

I will test it for him @dfiloni ;)

bqphone commented 7 years ago

sorry i missed dfiloni solution, will test it asap

Flohack74 commented 7 years ago

We will release this fix in 2.4.41 soon!

Flohack74 commented 7 years ago

Sorry latest Q/A shows its not solved right now...

bqphone commented 7 years ago

sadly the bug is not fixed yet with the new update.

Flohack74 commented 7 years ago

Hello,

I debugged now server/client communication, and it seems it´s some kind of bug on the server. While the user id is already reported as invalid, the server still sends user, dialog and message data for this stale dialog. The app does indeed delete the chat (it disappears for a moment), but then with the next update its back.

Will need to investigate how to still get rid of it ;)

Flohack74 commented 6 years ago

Hello, can you re-test this with beta 2.5.0 please? It seems to work now...

bqphone commented 6 years ago

Where can i get the beta version?

Flohack74 commented 6 years ago

https://github.com/ubports/telegram-app/releases/download/v2.5.0/com.ubuntu.telegram_2.5.0.0_armhf.click

bqphone commented 6 years ago

It is fixed, great job mate.

Flohack74 commented 6 years ago

You are welcome. But probably it was due to the API upgrade to 41, not because I fixed it hehe :)