-
Notifications
You must be signed in to change notification settings - Fork 17
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: wasm support for browser #204
feat: wasm support for browser #204
Conversation
update database impl update takewhile benchmark update basic operations adapt light-client-lib with sync storage update debug misc stuff update new tests
CCC fix released in ckb-devrel/ccc#93 |
(Mis-clicked to close, please ignore that) |
ckb-devrel/ccc#92 is the fix, and ckb-devrel/ccc#93 is the release of the fix. The fix was included in the changelog. |
I found it just now, thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rust part LGTM.
This PR adds support for running light client in browser, including the following major changes:
light-client
intolight-client-lib
,light-client-bin
, andlight-client-wasm
light-client-db-worker
to provide sync API access for IndexedDBlight-client-js
as a JS API wrapper for frontend developers, with type definition from@ckb-ccc/core
and similar APIs like RPC clientlight-client-wasm
,light-client-db-worker
andlight-client-js
, making them easy to be published to registries like npmThings need to be done:
Waiting for a bug in@ckb-ccc/core
fix:hexFrom
in@ckb-ccc/core/barrel
not producing valid hex strings when an empty array is provided ckb-devrel/ccc#90 to be fixedMost types we used from@ckb-ccc/core
are not exposed by default. Need more communication with devrel to discuss if these should be exposedIntegrated tests and unit-testsnpm package: https://www.npmjs.com/package/ckb-light-client-js