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

fix: deserialize block num as U64 #39

Merged

Conversation

onbjerg
Copy link
Member

@onbjerg onbjerg commented Nov 21, 2023

We need to be able to deser things like 0x1 to u64 but Rust does not do this by default, so we deser it as U64 intermittently and cast it to u64

Copy link
Contributor

@Evalir Evalir left a comment

Choose a reason for hiding this comment

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

reasonable

@onbjerg onbjerg merged commit 7862a03 into onbjerg/alloy-temp-provider-trait Nov 21, 2023
0 of 16 checks passed
onbjerg added a commit that referenced this pull request Nov 25, 2023
gakonst pushed a commit that referenced this pull request Dec 18, 2023
* wip

* chore: docs

* feat: add `get_block`

* fmt

* fix rebase

* more fix

* fmt

* more rebase fix

* more fix

* fmt

* rm unused crates

* more fix

* async trait wasm awesome hack

* ?

* ??

* ??????????

* hokus pokus

* feat: add tracing methods and a couple other ones (#32)

* chore: TransactionRequest -> CallRequest (#33)

* fix: return U256 for estimate_gas (#35)

* fix: serialize block num w/o leading zeros (#37)

* fix: use u64 for block numbers (#38)

* fix: serialize block num w/o leading zeros

* fix: use `u64` for block numbers

* fix: deserialize block num as U64 (#39)

* fix: typo

* fix: dont skip gas used ratio if empty (#40)

* workaround

* fix: actual fix

* fix: actual fix part 2

* fix(rpc-types): access list keys (#42)

* workaround

* fix: actual fix

* fix: actual fix part 2

* fix access list keys

---------

Co-authored-by: Enrique Ortiz <hi@enriqueortiz.dev>

* chore: remove Cow from `TempProvider` (#43)

* chore: start removing cows

* chore: uncow tempProvider

* Apply suggestions from code review

Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>

* switch to single element tuple

---------

Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>

* feat(`rpc-types`): RLP encoding/decoding for transaction types (#36)

* feat: add rlp encoding/decoding to tx types

* feat: add encodable/decodable traits to tx

* chore: remove out-of-scope func

* chore: remove bad links on comments

* chore: fix docs

* clippy

* feat(`TempProvider`): `raw_request` (#45)

* feat(): raw_request

* chore: remove unneeded async block

* chore: bump alloy-primitives

* chore: set uncle as default when missing transactions field

* fmt

* chore: remove pub

* feat: support for additional fields on TransactionReceipt and Block (#66)

* chore: add support for OtherFields in Transaction (#68)

* chore: turn off reqwest default features

* chore: address provider review comments (#87)

* chore: remove self: sync from defs

* chore: remove most generics

* chore: default for BlockId

* chore: remove unnecesary reassignment

* chore: auto impl on mut/rc

* chore: unnecesary assignment

---------

Co-authored-by: Enrique Ortiz <hi@enriqueortiz.dev>
Co-authored-by: DaniPopes <57450786+DaniPopes@users.noreply.github.com>
@DaniPopes DaniPopes deleted the onbjerg/get-block-number-fix branch March 11, 2024 16:12
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