-
Notifications
You must be signed in to change notification settings - Fork 22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[mobile] Multiple notifications generated upon Obsidian startup #149
Comments
Some further observations: the multi-notification on mobile seem to happen rarely, not in day-to-day use. Opening, switching, reopening a vault, closing, exiting, etc. don't cause the superfluous notifications flow. The screenshot attached to this ticket comes from scenario when the Obsidian app and the vault hasn't been open for a while (a month or so). At code level the notifications are generated only when the plugin is initialized by Obsidian. Multiple repeated notifications suggest some kind of auto-restarting of plugins subsystem by Obsidian. Hence a correct response of the plugin to Obsidian-generated events. Need to collect more observations - as for now this seem to be a minor issue occurring in some edge cases. |
I agree with you, it is not a breaking point. I'm not sure but opening my vault it seems frequent, only at opening. After that I can work without any other notification. I'll try to find a way to reproduce it. I'll keep you informed. Anyway @SebastianMC thanks a lot for you effort in maintaining the plugin. |
Did some more investigation and the final conclusion is to keep the behavior as-is.
Based on the above, closing the ticket. |
As a result of internal changes in Obsidian 1.6.x for plugin lifecycle and probably related to the bugfix #147 a series of notifications is shown by the plugin on start on mobile (screenshot below).
While it is better to have superfluous notifications and correct sorting applied automatically than no custom sorting applied, the multiple notifications deserve investigation and fixing.
First reported by @Azmoinal in this comment under #147
Screenshot for context

The text was updated successfully, but these errors were encountered: