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

v3.1.1 #918

Merged
merged 17 commits into from
Feb 5, 2024
Merged

v3.1.1 #918

merged 17 commits into from
Feb 5, 2024

Conversation

donavanbecker
Copy link
Collaborator

3.1.1 (2024-02-05)

What's Changed

Full Changelog: v3.1.0...v3.1.1

donavanbecker and others added 17 commits February 2, 2024 23:12
## ♻️ Current situation

The retry functionality introduced in #694 has no effect.

## 💡 Proposed solution

Revert the change to reduce complexity and use async/await.
Co-authored-by: Donavan Becker <beckersmarthome@icloud.com>
## ♻️ Current situation

The following error appears:
```
This plugin threw an error from the characteristic 'Hold Position': Characteristic Property 'minValue' can only be set for characteristics with numeric format, but not for bool. See https://homebridge.io/w/JtMGR for more info.
This plugin threw an error from the characteristic 'Hold Position': Characteristic Property 'maxValue' can only be set for characteristics with numeric format, but not for bool. See https://homebridge.io/w/JtMGR for more info.
This plugin threw an error from the characteristic 'Hold Position': Characteristic Property `minStep` can only be set for characteristics with numeric format, but not for bool. See https://homebridge.io/w/JtMGR for more info.
```

## 💡 Proposed solution

Remove the `setProps` call.
## ♻️ Current situation

If the curtain mode was not defined in the settings the following error
would occur:
```
Curtain: Curtain failed BLEpushChanges with BLE Connection, Error Message: "The type of running mode is incorrect: object"
```

## 💡 Proposed solution

- Set position mode to strings 
- Remove redundant code
- Set proper values for node-switchbot

Co-authored-by: Donavan Becker <beckersmarthome@icloud.com>
## ♻️ Current situation
The `require` function is not supported with ESM, leading the following
error:
```
ReferenceError: require is not defined
```

## 💡 Proposed solution

Use a dynamic import.
## ♻️ Current situation

Lint globbing doesn't work.

## 💡 Proposed solution

Fix glob pattern and reuse the existing npm script.
## [3.1.1](https://github.com/OpenWonderLabs/homebridge-switchbot/releases/tag/v3.1.1) (2024-02-05)

### What's Changed
- Fix BLE connection [#907](#907), Thanks [@dnicolson](https://github.com/dnicolson)
- Fix BLE curtain mode [#908](#908), Thanks [@dnicolson](https://github.com/dnicolson)
- Fix curtain characteristic warning [#909](#909), Thanks [@dnicolson](https://github.com/dnicolson)
- Revert curtain retry functionality [#911](#911), Thanks [@dnicolson](https://github.com/dnicolson)
- Code Cleaup [#910](#910) [#912](#912) [#913](#913)  [#917](#917), Thanks [@dnicolson](https://github.com/dnicolson)
- Housekeeping and updated dependencies.

**Full Changelog**: v3.1.0...v3.1.1
@donavanbecker donavanbecker merged commit f0063b1 into latest Feb 5, 2024
5 checks passed
@donavanbecker donavanbecker deleted the beta-3.1.1 branch February 5, 2024 17:34
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