Feat: Enable support for snoo sleepytime timeout levels #46
Conversation
This file contains hidden or 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
There was a problem hiding this comment.
Pull request overview
This PR adds support for configurable timeout levels for the Snoo's sticky white noise feature. Previously, the timeout was hardcoded to 15 minutes; now users can select from timeout values ranging from 5 to 180 minutes in 5-minute increments.
Changes:
- Added a new
SnooNoiseTimeoutLevelsIntEnum with timeout options matching the official Snoo app - Updated
set_sticky_white_noise()method to accept an optional timeout_value parameter with a default of 15 minutes - Refactored imports in baby.py from absolute to relative imports
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| python_snoo/containers.py | Introduces SnooNoiseTimeoutLevels IntEnum with timeout values from 5-180 minutes; adds from __future__ import annotations import |
| python_snoo/snoo.py | Updates set_sticky_white_noise() to accept optional timeout_value parameter and imports the new enum |
| python_snoo/baby.py | Converts imports from absolute to relative format (unrelated refactoring) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Created boilerplate to handle sending a timeout value through to the API with the same values as the snoo app provides. Default value set to 15 minutes if no timeout is provided