-
Notifications
You must be signed in to change notification settings - Fork 1
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
Initial Implementation #2
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initial Implementation * Make core package string configurable Prevent raising when package not found * Update intent registration to use decorators
* Update LICENSE.md and file headers to BSD-3 * Update license headers Update README formatting Add updated skill.json * Update tags and skill.json to identify Neon Original skills * Update license formatting * Add libpulse dep for holmesV dependency compat.
Co-authored-by: Daniel McKnight <daniel@neon.ai>
* Add skill.json automation * Update skill.json * fix neon-utils version spec Co-authored-by: Daniel McKnight <daniel@neon.ai> Co-authored-by: NeonDaniel <NeonDaniel@users.noreply.github.com>
* Prep for PyPI release Update for neon-utils 1.0.0 compatibility * Update skill.json * Fix references in skill tests * Update unit test to match resource filename refactor Co-authored-by: Daniel McKnight <daniel@neon.ai> Co-authored-by: NeonDaniel <NeonDaniel@users.noreply.github.com>
Co-authored-by: Daniel McKnight <daniel@neon.ai>
* Interface with PHAL plugin to check for and initiate core updates * Update skill.json * Cleanup dependencies * Update skill.json * Always check latest release on update intent Format parsed version into something pronounceable * Update intents Update unit tests Add intent tests * Install skill package in unit test automation * Install skill package in other unit test automation * Add skill resource tests * Troubleshooting core version checks * Update to handle modified plugin API * Fix typo in updated version check * Troubleshoot intent test failures * Troubleshooting Padatious intent compatibility * Update skill.json * Add setting to include prereleases * add phal plugin to requirements * Update skill.json Co-authored-by: NeonDaniel <NeonDaniel@users.noreply.github.com>
* Add intent to update configuration with updated tests * Add more intents with tests
* Add intent to create new boot media with methods to interact with PHAL plugin that does actual image creation * Update spoken dialog for clarity * Update tests to cover new intent * Resolve intent test failures * Fix typo in confirmation dialog * Add spoken confirmation of image installation start Move notification titles to resources * Clean up test file Update skill documentation * Update skill.json * Update `update_device.intent` to support example utterance * Refactor bus events for consistency * Dismiss notification on interaction * Troubleshooting notification dismissal * Fix up notification dismissal * Make download complete notification transient * Troubleshooting notification dismissal * Cleanup notification dismissal Add controlled notification while image is being written * Add notification content to dialog file * Fix bug in notification dismissal Message data * Starting unit tests for added methods * Finish unit tests Update docstrings * Fix event handling in unit tests Co-authored-by: NeonDaniel <NeonDaniel@users.noreply.github.com>
* Add a check on init for available updates and notify user * Limit update notifications to checks on `ready` Dismiss update notification on update start * Fix typo in message data check to dismiss notification on update start * Add option to disable update notifications in skill settings * Update settingsmeta.yml with referenced settings
NeonDaniel
reviewed
Jan 20, 2023
* Update skill requirements Update versions in automation * Update skill.json Co-authored-by: NeonDaniel <NeonDaniel@users.noreply.github.com>
NeonDaniel
approved these changes
Jan 20, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Initial implementation with PHAL Plugin integration
Includes intents to update core, update config, and create new media
Integrates with https://github.com/NeonGeckoCom/neon-phal-plugin-core-updater for core updates
Integrates with https://github.com/NeonGeckoCom/neon-phal-plugin-reset for config updates and media creation