-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Extensions: Added uninstall and update process #27768
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
base: master
Are you sure you want to change the base?
Conversation
f5ccd82
to
e371e3d
Compare
Thanks for the fast review @cbjeukendrup! I added a commit to address your suggestions. |
src/framework/extensions/qml/Muse/Extensions/internal/EnablePanel.qml
Outdated
Show resolved
Hide resolved
@Kavakuo Thank you for your improvement! A few suggestions:
|
2ecbd56
to
b18fbd7
Compare
@igorkorsukov Thanks for your feedback! I mostly changed the things as you suggested.
Since only extensions in the new format can be "installed", I first thought that the
Instead of |
Hi all,
I recently played around with the new extension format (
.mext
) and noticed that it is currently not possible to ship updates for an extension without performing a manual update process (deleting the folder, drag&drop the extension for installation). I didn't find any issues for this, so I thought this could be a small fun weekend project. So this PR introduces two new features improve the process:I am no C++ expert, so please let me know if you see any room for improvements that I should change.
Remaining ToDos: