Skip to content

Support more commands, reorder, raise error with screenshot and camshot#25

Merged
cgarwood merged 2 commits into
cgarwood:masterfrom
infeeeee:master
Aug 30, 2024
Merged

Support more commands, reorder, raise error with screenshot and camshot#25
cgarwood merged 2 commits into
cgarwood:masterfrom
infeeeee:master

Conversation

@infeeeee

@infeeeee infeeeee commented Aug 23, 2024

Copy link
Copy Markdown
Contributor

Thank you for this package.

Added some missing commands.

Commands used to be in the order of contributions. I reordered them similar to official docs, and grouped them by similarity. Much easier to read the code this way.

getCamshot and getScreenshot now raises an error just like any other commands.

Tested this with my tablet running "Fully Single App Kiosk" .

Supersedes #18

@cgarwood cgarwood left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks! One comment on the start/stop daydream commands, but otherwise looks good.

Comment thread fullykiosk/__init__.py Outdated
@infeeeee infeeeee requested a review from cgarwood August 29, 2024 22:36

@cgarwood cgarwood left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thanks!

@cgarwood cgarwood merged commit a21769b into cgarwood:master Aug 30, 2024
@neubi4

neubi4 commented Feb 15, 2026

Copy link
Copy Markdown

Hi @cgarwood

is it possible to release a new version of the library? I would like to use the new triggerMotion command from this PR,, but there is no new release after this merge.

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.

3 participants