[feat]: useBluetooth #224
[feat]: useBluetooth #224
Conversation
| // eslint-disable-next-line @typescript-eslint/no-redundant-type-constituents | ||
| const [device, setDevice] = useState<BluetoothDevice | undefined>(undefined); | ||
| // eslint-disable-next-line @typescript-eslint/no-redundant-type-constituents | ||
| const [server, setServer] = useState<BluetoothRemoteGATTServer | undefined>(undefined); |
There was a problem hiding this comment.
По поводу всех дизейблов, кроме error, вроде и без них все работает, так что можно их убрать?
There was a problem hiding this comment.
Да, в этом случае работает без проблем, просто неприятно, когда линтнер ошибку светит
| // eslint-disable-next-line @typescript-eslint/no-redundant-type-constituents | ||
| const [server, setServer] = useState<BluetoothRemoteGATTServer | undefined>(undefined); | ||
| // eslint-disable-next-line @typescript-eslint/no-redundant-type-constituents | ||
| const [error, setError] = useState<unknown | null>(null); |
There was a problem hiding this comment.
По поводу самой ошибки, почему именно unknown? Ты же можешь проверить, что именно падает и сделать это строкой, если не получается, то указать кастомную ошибку, условно setError('error') или заменить на булевую ошибку, чтобы просто обозначать, что она есть. Либо прокинуть onError, чтобы передавать пользователю ошибку, сам ты все равно её не используешь. Либо вообще ничего не делать и кидать ошибку в консоль, но тут хз.
There was a problem hiding this comment.
Мерси за ревью, поправил ошибку)
| } = options || {}; | ||
|
|
||
| const [isSupported, setIsSupported] = useState<boolean>(false); | ||
| const [isConnected, setIsConnected] = useState<boolean>(false); |
There was a problem hiding this comment.
Еще, кстати, можно убрать <boolean>, ты же явно указываешь начальное значение
Привет ребят, столкнулся с проблемами при типизации блютуза и ошибок. Соответственно использовал eslint-disable-next-line. Буду рад ревью и постараюсь исправить