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
Enhance(UX): marketplace #9609
Enhance(UX): marketplace #9609
Conversation
…installed plugins
(set-enabled! t) | ||
(plugin-handler/set-enabled-auto-check-for-updates t) | ||
(notification/show! | ||
[:span text [:strong.pl-1 (if t "ON" "OFF")] " !"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the space before "!" is not needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QA'd works as expected 🎉 Been running this PR for a few days in my main Logseq build. Tested the plugin update feature by pushing an update to the Logseq Nord theme.
- disable check auto update (wait)
- push an update to a plugin
- enable auto-check for updates (wait)
- notification to update the plugin appears.
I did not close/restart the instance during the test to ensure it could change state while the app is open. I didn't measure the wait time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
QAed, LGTM
…installed plugins
5e2fae5
to
c1cdf2a
Compare
TODOs
fix #9581
CleanShot.2023-06-09.at.09.56.15.mp4