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

🚀 3.14.0 #239

Merged
merged 11 commits into from
Oct 22, 2023
Merged

🚀 3.14.0 #239

merged 11 commits into from
Oct 22, 2023

Conversation

* Generating CSS from SASS.

* Splitting the sources.

* Making style comment removable.

* Upgrading all dependencies.
@RobinTail RobinTail added refactoring Better implementation of the same thing release Chores for releasing new versions dependencies labels Oct 17, 2023
@coveralls
Copy link

coveralls commented Oct 17, 2023

Coverage Status

coverage: 100.0%. remained the same when pulling 677ce66 on develop into d618fd9 on master.

* Drop Python 3.7 (EOL).

* Type constraints for the list of Tasks.

* Add mypy config.

* Using MyPy in CI.

* Revert "Drop Python 3.7 (EOL)."

This reverts commit bee43c7.

* Add final line to mypy config.

* Ignore missing types of flask.

* Draft for Model type.

* Introducing the correctly typed initial state for the model.

* Add final newline.

* Listing the dependency on typing-extensions.

* Ensure presence of polling timer before cancelling it in on_shutdown method.

* Disabing pylint public methods limit for model.

* Ref: naming.

* Ref: exposing final Model type and get_initial_model() helper for whole one.

* Minor: encoding header.

* Ref: type constraints in update_ui for upcoming tasks.

* Type constraints for get_upcoming_tasks() method.

* Ref: improving types for get_upcoming_tasks() argument subjects.

* Run type checking before linting.

* Minor: excluding snapshots from coverage reporting.

* Update octoprint_octorelay/model.py
@RobinTail RobinTail added the coverage Improving the tests label Oct 17, 2023
* Moving response preparation outta handlers.

* Making handle_cancel_task_command to return bool.

* Adjusting the tests.

* Updating tests descriptions.

* Testing the faulty case.

* Introducing class HandlingException for using in API handlers.

* Restoring original behavior to deprecate.

* Ref: moving UI update outta try block.

* Fix: invalid relay index should also be 400 and have deprecated response.

* Moving the disabled relay get status handling into on_api_command method.

* Update octoprint_octorelay/__init__.py

* Update tests/test_init.py

* Apply suggestions from code review

* Update tests/test_init.py
@RobinTail RobinTail added the enhancement New feature or request label Oct 21, 2023
@RobinTail RobinTail changed the title 🚀 Next 🚀 3.14.0 Oct 21, 2023
@RobinTail
Copy link
Collaborator Author

🚀 3.14.0rc1

@RobinTail
Copy link
Collaborator Author

✅ Regression test passed
✅ API test passed
✅ QA passed

* Moving required params checking into the on_api_command() method.

* DNRY: reducing complexity.
* Introducing version | v parameter.

* Consistent arguments of API v2.

* Fix linting issues.

* Changed response for cancelTask command in v2.

* getStatus command response remains.

* Fixing mypy issues.

* Changing response for update command of v2.

* Changing the response for listAllStatus command of v2.

* Fix linting issue.

* Partial revert: keeping id naming.

* Minor: readability.

* Ref: readbility for getStatus command branching.

* Update octoprint_octorelay/__init__.py

* Ref: Ensure int for version

* Ref: todo for subject parameter name for next major release

* Type constraints for listing command.

* Fix linting issue.

* UI: Switching to API v2.
@RobinTail
Copy link
Collaborator Author

🚀 3.14.0rc2

@RobinTail
Copy link
Collaborator Author

✅ Regression test passed

@RobinTail
Copy link
Collaborator Author

RobinTail commented Oct 22, 2023

⚠️ reason property is lost from update command response payload in case of disabled relay

@RobinTail
Copy link
Collaborator Author

✅ legacy API tests passed
✅ featured API v2 tests passed
✅ QA passed

@RobinTail RobinTail marked this pull request as ready for review October 22, 2023 14:41
README.md Outdated Show resolved Hide resolved
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.

Ready

@RobinTail RobinTail linked an issue Oct 22, 2023 that may be closed by this pull request
@RobinTail RobinTail merged commit e512dd4 into master Oct 22, 2023
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
coverage Improving the tests dependencies enhancement New feature or request refactoring Better implementation of the same thing release Chores for releasing new versions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

API inconsistency and payload props naming
2 participants