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.

Sign up for GitHub

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

Jump to bottom

feat(grpc,node-wasm): add javascript bindings for tx client #510

Merged
merged 60 commits into from
Jan 27, 2025
Merged

feat(grpc,node-wasm): add javascript bindings for tx client #510

merged 60 commits into from
Jan 27, 2025

Conversation

zvolin
Copy link
Member

@zvolin zvolin commented Jan 14, 2025

No description provided.

fl0rek and others added 30 commits November 28, 2024 09:39
@fl0rek
feat(node-wasm, types)!: Add blob service for wasm
999138b
@fl0rek
Merge remote-tracking branch 'upstream/main' into feat/blob_service
55c5c5b
@fl0rek
fix wasm ci
a8a1b20
@fl0rek
new rust, more CI
8307ddc
@fl0rek
fix unused
795e6ac
@fl0rek
Merge remote-tracking branch 'upstream/main' into feat/blob_service
a9d64d6
@fl0rek
increase timeout
4d2494c
@fl0rek
better tests, et al
bcfefca
@fl0rek
missed
69fb3e0
@fl0rek
wip
7dbfe9a
@fl0rek
fix commitment
5d5ac5a
@fl0rek
misc ci
e9e1ae0
@zvolin
feat(grpc)!: enable celestia-grpc usage within wasm
82dc2cb
@zvolin
remove env_or helper
99b3af1
@zvolin
remove unused import
e83598d
@zvolin
remove anyhow in tests
5cebaf8
@fl0rek
wip
9c6493c
@fl0rek
PR review, align to current wasm-bindgen
6a7b71c
@fl0rek
Merge branch 'main' into feat/blob_service
09e4cbb
@fl0rek
Add relevant getters to namespace
ab3cff9
@fl0rek
whoops didnt mean to commit that
5ec36f8
@zvolin
add transaction client
7079948
@zvolin
Merge branch 'main' into feat/grpc-wasm-support
b04e880
@zvolin
remove sleep from tests
74e461d
@zvolin
remove unused deps
3feb07f
@zvolin
rename shares_count to shares_len
66bef5b
@zvolin
add debug impls
6b6b820
@zvolin
fix blob shares_len doctest
201f091
@zvolin
match errors in tests
138fec1
@zvolin
wip
47c9f00
@zvolin zvolin requested review from oblique and fl0rek January 14, 2025 14:42
oblique
oblique reviewed Jan 17, 2025
View reviewed changes
Copy link
Member

@oblique oblique left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!!

grpc/Cargo.toml Outdated Show resolved Hide resolved
grpc/src/grpc/auth.rs Show resolved Hide resolved
fl0rek
fl0rek reviewed Jan 20, 2025
View reviewed changes
cli/js/index.js Outdated Show resolved Hide resolved
grpc/src/grpc/auth.rs Show resolved Hide resolved
grpc/src/js_client.rs Outdated Show resolved Hide resolved
grpc/src/js_client.rs Outdated Show resolved Hide resolved
grpc/src/js_client.rs Outdated Show resolved Hide resolved
grpc/src/js_client.rs Outdated Show resolved Hide resolved
grpc/src/js_client.rs Outdated Show resolved Hide resolved
grpc/src/js_client.rs Show resolved Hide resolved
cli/js/index.js Show resolved Hide resolved
@zvolin zvolin mentioned this pull request Jan 21, 2025
Closed
@zvolin zvolin linked an issue Jan 21, 2025 that may be closed by this pull request
Lumina has no way to submit transactions/blobs from the browser #425
Closed
@zvolin zvolin mentioned this pull request Jan 21, 2025
Open
@zvolin @fl0rek
florek's grammars
235868c 
Co-authored-by: Mikołaj Florkiewicz <[email protected]>
Signed-off-by: Maciej Zwoliński <[email protected]>
@zvolin zvolin mentioned this pull request Jan 21, 2025
Open
@zvolin zvolin requested a review from fl0rek January 21, 2025 13:48
fl0rek
fl0rek approved these changes Jan 22, 2025
View reviewed changes
Copy link
Member

@fl0rek fl0rek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks Good To Me!

@oblique oblique merged commit a2213af into eigerco:main Jan 27, 2025
29 checks passed
This was referenced Jan 27, 2025
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fl0rek fl0rek fl0rek approved these changes

@oblique oblique oblique approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Lumina has no way to submit transactions/blobs from the browser
3 participants
@zvolin @oblique @fl0rek