search

thegecko / web-bluetooth-dfu

Device firmware update with Web Bluetooth
https://thegecko.github.io/web-bluetooth-dfu/
MIT License
96 stars 41 forks source link

nRF51 Support #49

Closed septs closed 4 years ago

septs commented 4 years ago

see https://developer.nordicsemi.com/nRF5_SDK/nRF51_SDK_v4.x.x/doc/html/group__dfu__ble__service__spec.html

thegecko commented 4 years ago

This supports nRf51 with softdevice S130

septs commented 4 years ago

Screenshot_20200905-203417_nRF_Connect

the target device DFU use 00001530-1212-efde-1523-785feabcd123

thegecko commented 4 years ago

I assume the default has changed, overwrite it here: https://github.com/thegecko/web-bluetooth-dfu/blob/d050b89497ae297829a772788e0c052781f4b945/src/secure-dfu.ts#L458

septs commented 4 years ago

OK.

septs commented 4 years ago 
ERROR in ./src/components/SensorConsole/dfu/index.ts
Module not found: Error: Can't resolve 'web-bluetooth-dfu' in '/Users/septs/Projects/mops-vida-pm-watchdog/src/components/SensorConsole/dfu'
 @ ./src/components/SensorConsole/dfu/index.ts 2:0-42 6:20-29 7:25-34
 @ ./src/components/SensorConsole/FirmwareVersion.tsx
 @ ./src/components/SensorConsole/index.tsx
 @ ./src/Entry.tsx
 @ ./src/index.ts
septs commented 4 years ago

https://github.com/thegecko/web-bluetooth-dfu/blob/5cdb47563df3d2188d0f1051b9954022f56dcbd6/package.json#L10-L12 this syntax on webpack may not be supported.

Reference: https://webpack.js.org/configuration/resolve/

septs commented 4 years ago

@thegecko

thegecko commented 4 years ago

This is an old repo and wasn't designed with webpack in mind. PRs welcome.

septs commented 4 years ago

@thegecko do i allow many changes? e.q: upgrade all packages migrate tslint to eslint use prettier format code

and some api break change.

etc.

thegecko commented 4 years ago

Please keep separate PRs focussed on a single area

septs commented 4 years ago

Sorry, I decided to write a more modern https://github.com/NiceLabs/nordic-dfu