ferdium / ferdium-app

All your services in one place, built by the community
https://ferdium.org
Apache License 2.0
2.68k stars 164 forks source link

[Bug]: No notifications or unread messages icon showing for Telegram (Z version) #1009

Open PlaceboPRS opened 2 years ago

PlaceboPRS commented 2 years ago

Preflight Checklist

Ferdium Version

6.0.0 Nightly 23

Recipe Name

Telegram

Recipe Version

3.3.0

Last Known working version of the recipe

No response

Steps to reproduce

  1. Install Telegram recipe
  2. login to telegram
  3. have people message you
  4. wonder why there's no notifications/unread message icons

Expected Behavior

I expect to receive notifications and/or see an unread message icon

Actual Behavior

I don't receive notifications and/or see an unread message icon

Screenshots

image

Additional Information

No response

vraravam commented 2 years ago

Unable to reproduce this - I use Telegram, and I can see unread counts

mydarkstar commented 2 years ago

I also have trouble seeing an unread badge on the Telegram Web 'Z' version. They seem to work fine with the 'K' version, however. @PlaceboPRS, could it be that you're using the 'Z' version?

I also pointed out that letting Telegram randomly choose one of the two versions can be detrimental to the user experience a bit ago here.

Besides, all of your chats seem to be muted. Therefore you shouldn't be shown an unread badge anyway, no?

PlaceboPRS commented 2 years ago

Yes I was using Z because K wastes so much space:

image

I don't let Telegram randomly choose versions I chose the one that used the space best ;)

And no of course all my chats are not muted, that would be very dumb to mute all chats then wonder why I'm not getting notifications! I have a bunch of group chats muted but all my individual chats aren't muted :)

mydarkstar commented 2 years ago

Alright! I looked into this issue, and it seems like the following lines are broken on the 'WebZ' version: https://github.com/ferdium/ferdium-recipes/blob/18801ed0c02627e87639dc0848cab44dacc18be2/recipes/telegram/webview.js#L8-L10

The returned value from this query is 'Telegram Web' on the 'WebZ' version, while the script expects the string to contain WebZ. On the 'WebK' version the query still works, as it returns 'Telegram WebK'.

The query could be changed to check for WebK and assume it's WebZ otherwise. Another way of detecting the version may be using document.location.pathname, which should be either /z/ or /k/. This approach is not as elegant, though.

PlaceboPRS commented 2 years ago

Hi, sorry but this isn't fixed, I'm using the Telegram recipe 3.3.1 and with Z version I'm still not getting notifications/unread counts.

vraravam commented 2 years ago

reopening

vraravam commented 2 years ago

Though I reopened this ticket, I have to say that I use Telegram, and I get notifications and unread counts. im on macos and the K version of Telegram

PlaceboPRS commented 2 years ago

Though I reopened this ticket, I have to say that I use Telegram, and I get notifications and unread counts. im on macos and the K version of Telegram

Yes K version gets notifications, Z version doesn't. I use Z because K badly wastes/uses space.

I'm using Windows 11.

mydarkstar commented 2 years ago

Heya, on recipe v3.3.3, Telegram Web Z has been showing notification badges properly.

For groups and channels, it adds a blue dot: screenshot1

For DMs, it shows a number: screenshot2

Muted chats are ignored. This issue does not seem to persist on either the K or Z versions when using Ferdium Beta 2! Please let us know if this is still an issue to you, @PlaceboPRS. Otherwise, I'd say we can close this.

jasmines commented 2 years ago

Unread count stopped working today, with 6.0.0 nightly 80

ilopmar commented 2 years ago

Telegram notifications were working until a few days ago. I updated to nightly.80 and they are not displayed when using Telegram K. If I switch to Telegram Z they are displayed. I've tried also with beta2 and the behaviour is the same :(

P.S: I'm running Linux Mint with Cinnamon desktop.

jasmines commented 2 years ago

how can I switch to Z?

ilopmar commented 2 years ago

You can do it from the "hamburger" menu:

image

The problem is that you can't set that version as default. At least I don't know how to do it.

MadelineProtoFan commented 2 years ago

Sound notifications do not work even in Z version

MadelineProtoFan commented 2 years ago

Telegram notifications were working until a few days ago. I updated to nightly.80 and they are not displayed when using Telegram K. If I switch to Telegram Z they are displayed. I've tried also with beta2 and the behaviour is the same :(

P.S: I'm running Linux Mint with Cinnamon desktop.

The same. 2 days ago everything was working :(

MadelineProtoFan commented 1 year ago

Notifications are showing. But there is no sound for them :( Using the latest 101 version :( Do not have sound both K and Z version (everywhere the sound in on)

MadelineProtoFan commented 1 year ago

I can send the money to the developer. Please contact me and i will send the money for fixing sound notifications in telegram web! I cannot work without them

MadelineProtoFan commented 1 year ago

Please fix sound notifications in telegram. A month past

MadelineProtoFan commented 1 year ago

Today after update K version started to send sound notifications! Everything is working. It was not a Ferdium problem. Telegram team fixed the sound notification problem WHAT A GOOD DAY :)

SpecialAro commented 1 year ago

Hello @PlaceboPRS, is this issue still active?

carlosala commented 1 year ago

At least in Z version, this is still happening. No notifications at all. K version works as expected

carlosala commented 1 year ago

And counter in Z version is not correct, it's showing random numbers./

RomLecat commented 1 year ago

Hi,

I still get this issue (no red dot on Z version, recipe v3.3.5), and this is annoying because I can't switch to K version due to #859.