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

feat: add bioweapon mode as fan mode #965

Merged
merged 18 commits into from
May 29, 2024
Merged

feat: add bioweapon mode as fan mode #965

merged 18 commits into from
May 29, 2024

Conversation

dandelionclock
Copy link
Contributor

@dandelionclock dandelionclock commented May 11, 2024

Added bioweapon mode as fan mode. Bioweapon mode can be turned on independently of other preset modes such as dog mode or camp mode, so it is better to include it as fan mode instead of one of the preset modes.
Requires zabuldon/teslajsonpy/pull/468

dandelionclock and others added 8 commits May 10, 2024 19:19
Bioweapon mode can be turned on/off independently from other preset modes like Camp mode or Dog mode. By using fan mode, you can have both Bioweapon mode and Camp mode turned on at the same time.

Requires commits to teslajsonpy to be merged to funcion.
@alandtse
Copy link
Owner

Please handle the teslajsonpy bump and we can merge this in.

@dandelionclock
Copy link
Contributor Author

I updated the files with teslajsonpy 3.11.0. Not too sure about poetry.lock file but I updated the version and the hash manually

@alandtse
Copy link
Owner

You need to run poetry update.

@alandtse
Copy link
Owner

Thanks. Please take a look at the failing test. You can ignore the precommit error since that looks like something unrelated to climate.

@dandelionclock
Copy link
Contributor Author

I think I got it? Thank you for your patience.

@alandtse alandtse merged commit a13f717 into alandtse:main May 29, 2024
6 of 7 checks passed
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.

2 participants