Skip to content

Support power saving service#255

Open
Ruben-Lohberg wants to merge 2 commits into
mainfrom
auto-off
Open

Support power saving service#255
Ruben-Lohberg wants to merge 2 commits into
mainfrom
auto-off

Conversation

@Ruben-Lohberg
Copy link
Copy Markdown
Collaborator

No description provided.

@Ruben-Lohberg Ruben-Lohberg linked an issue May 18, 2026 that may be closed by this pull request
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 18, 2026

CLA assistant check
All committers have signed the CLA.

Comment thread open_wearable/pubspec.yaml Outdated
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

PR Build Artifacts

Full workflow run: https://github.com/OpenEarable/app/actions/runs/26561140137

@Ruben-Lohberg
Copy link
Copy Markdown
Collaborator Author

Checks fail because the required version of the flutter library is not yet merged.

@Ruben-Lohberg
Copy link
Copy Markdown
Collaborator Author

@DennisMoschina please review. The code is mostly generated by Codex

@github-actions
Copy link
Copy Markdown
Contributor

Visit the preview URL for this PR (updated for commit 91bf31b):

https://open-earable-web--pr255-auto-off-mtvtnufb.web.app

(expires Thu, 25 Jun 2026 08:01:35 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: c7397c11177c71d8d81172cea9365829823fb41c

Copy link
Copy Markdown
Collaborator

@DennisMoschina DennisMoschina left a comment

Choose a reason for hiding this comment

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

Looks good so far. Only comment I have is how differing modes in stereo devices are handled. For the microphone and audio mode, we display the mode for each device instead of merging it into the same mode automatically. @TobiasRoeddiger do we want the same behaviour for the power saving mode or is the approach with merging into the same mode acceptable?

@TobiasRoeddiger
Copy link
Copy Markdown
Member

yes please do not merge them automatically

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

App: add configurable auto-off feature for earable devices

4 participants