search

estroBiologist / pluralchum

PluralKit integration for BetterDiscord
MIT License
41 stars 11 forks source link

PK tag doesnt display until hovered over #6

Closed reticivis-net closed 10 months ago

reticivis-net commented 2 years ago

not sure if this is an issue on my end or what this plugin modifies the exact thing that you modify but it works instantly so i'd have a look at that i'll try to learn bdapi and implement some of these myself but for now ill just give it to you lol

estroBiologist commented 2 years ago

The major limitation here is that PC needs to wait for data from the PK server to modify messages, which happens asynchronously. But as far as I know, once that data arrives it should be able to patch messages instantly, including the PK tags.

Does the problem persist if you switch back and forth between channels?

reticivis-net commented 2 years ago

Hmmm it stopped happening, I'll have to look into this more to find a way to reliably reproduce it

estroBiologist commented 2 years ago

doesn't sound like a bug, just intended (if admittedly kind of unintuitive) behaviour. PChum caches member data, so proxy patching gets more consistent over time. there's a Clear Cache button in the plugin settings if you want to reproduce the behaviour, but with my current (read: bad) understanding of React, i don't think there's much i can do about it. i'll take another stab at it for 1.1, though.

ariagivens commented 10 months ago

Fixed by #32