Skip to content
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

JS: notification stub in sources #132

Merged
merged 13 commits into from
Jul 30, 2023
Merged

JS: notification stub in sources #132

merged 13 commits into from
Jul 30, 2023

Conversation

RobinTail
Copy link
Collaborator

@RobinTail RobinTail commented Jul 30, 2023

Closes #115
Continuation of #129 — QA revealed a need for a stub

⚠️ Breaking
Switching to ZIP-package distribution requires user to update the plugin TWICE.
After the first update there will be no control buttons in navbar which I consider a disruption of service, requiring a user's action to fix.

In order to minimise the confusion I'm making a stub showing a warning icon for attracting user's attention on the needed action, and a dialog explaining what to do.

@RobinTail RobinTail added documentation Improvements or additions to documentation enhancement New feature or request release Chores for releasing new versions labels Jul 30, 2023
@RobinTail RobinTail linked an issue Jul 30, 2023 that may be closed by this pull request
@coveralls
Copy link

coveralls commented Jul 30, 2023

Coverage Status

coverage: 100.0%. remained the same when pulling 5ba3485 on issue-115-stub into 36984a2 on develop.

@RobinTail
Copy link
Collaborator Author

Perhaps this should be marked as breaking.

@RobinTail RobinTail added the breaking Breaking changes label Jul 30, 2023
@RobinTail RobinTail marked this pull request as ready for review July 30, 2023 20:23
Copy link
Collaborator Author

@RobinTail RobinTail left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be fine

@RobinTail RobinTail merged commit 8216d27 into develop Jul 30, 2023
13 checks passed
@RobinTail
Copy link
Collaborator Author

RobinTail commented Jul 30, 2023

✅ QA passed

Assets for changelog

Screenshot 2023-07-30 at 22 39 11 Screenshot 2023-07-30 at 22 40 30

@RobinTail RobinTail mentioned this pull request Jul 30, 2023
RobinTail added a commit that referenced this pull request Jul 30, 2023
* Distributable ZIP-package (#129)

* Removing redundant env var in CI.

* Add sample releasing steps until the artifact upload.

* Add missing dependency installation step.

* Including changelog into the manifest.

* Add integration test.

* Removing JS file from the repo.

* Removing the changes checking step from UI CI.

* Add miniver installation step.

* Add versioning step.

* Installing miniver using GitHub Codespace.

* Ignoring some files.

* Using module name for importing version.

* Add EOL.

* Adding the miniver lines to setup.py.

* Add Wheels installation step.

* CI: excluding _version.py from coverage report.

* Changing the trigger for Release workflow.

* Trying checking with depth 0.

* Only trigger on published type of release.

* Patching miniver for using non-annotated tags.

* Add the asset attachment job.

* Using specific version of get-release action.

* Changing the package download link for pip.

* EOL for .gitattributes

* minor, empty line in setup.py

* Fix padding in release workflow.

* Readme: changing the latest ditribution link.

* Revert "Trying checking with depth 0."

This reverts commit 1f4a92c.

* Extracting versions into env, testing installation against variety.

* Fix: add wheel installation step to integration test.

* Minor: title.

* Fix: build job name.

* Ref: organizing the steps: UI first, Python next.

* Changelog: the future 2.3.0.

* JS: notification stub in sources (#132)

* Making a stub for installations from sources.

* Minor: grammar.

* Ref: moving show() to the end. Using cancel button of the dialog instead.

* Ref: shorter title.

* Release workflow: stub removal explicit step.

* Minior: comment.

* Removing color - looks too agressive.

* Fix: dialog notice font adjustments.

* Changelog: the future 3.0.0.

* Minor: text.

* Changelog: moving breaking changes to the top.

* Minor: text.

* Ref: keeping off-on near.

* Changelog: minor fixes and illustration.

* FIX: wrong latest release asset URL.
@RobinTail RobinTail deleted the issue-115-stub branch July 31, 2023 21:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Breaking changes documentation Improvements or additions to documentation enhancement New feature or request release Chores for releasing new versions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Switch to ZIP-package distribution channel
2 participants