callmearta / clubhouse-desktop

An unofficial Clubhouse desktop client
Other
212 stars 38 forks source link

[Bug] --- Notifications not loading #47

Closed vivekkj123 closed 3 years ago

vivekkj123 commented 3 years ago

Hi there,

Thank you for this wonderful unofficial clubhouse client. But there is a small bug on this application. The notification section is keep loading, but doesn't show anything. I've attached the screenshot below. Expecting a quick fix.

Screenshot showing the bug

Message from console

Message from console

Thanks, Vivek K J

subins2000 commented 3 years ago

Can confirm this bug. My output : image

subins2000 commented 3 years ago

Here's a small script to read the notifications from that object stored in temp (do this in browser console) :

Object.keys(temp1.notifications).forEach((k) => {
  o = temp1.notifications[k]
  if ('user_profile' in o)
    console.log(o.message + " - " + o.user_profile.name)
  else
    console.log(o.message)
})
callmearta commented 3 years ago

The problem is in some notification objects which don't posses a user_id. Feel free to do a PR if you managed to fix it. I don't have time rn to do so.

callmearta commented 3 years ago

Latest PR merged and the problem now should be gone. Closing this for now

stultus commented 3 years ago

I'm still getting this issue in the latest version

image