search

shady2k / logseq-inbox-telegram-plugin

MIT License
78 stars 5 forks source link

invertMessagesOrder isn't working when inboxName is null #42

Closed goldan closed 1 month ago

goldan commented 1 month ago

Hi, thank you for such a useful plugin.

Bug report: when inboxName is set to null, invertMessagesOrder setting is not respected: messages are added to the top of my daily page, not to the bottom. Restarting the plugin didn't help. Would it be possible to fix this?

shady2k commented 1 month ago

Can you check v1.0.5? Did it fix this issue?

goldan commented 1 month ago

Not really. Here is what I see in 0.1.5, after adding test and test2 messages:

image
goldan commented 1 month ago

I was able to fix it locally, please see my fork here: https://github.com/shady2k/logseq-inbox-telegram-plugin/compare/master...goldan:logseq-inbox-telegram-plugin:master But I think the code needs to be refactored before merging.