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

feat!: use more general topic representation for eth_getLogs #165

Merged
merged 4 commits into from
Feb 5, 2024

Conversation

rvanasa
Copy link
Collaborator

@rvanasa rvanasa commented Feb 2, 2024

Changes the Candid type of the topics field to vec vec text instead of vec text.

For example, vec {"0xabc", "0x123"} is now equivalent to vec {vec {"0xabc"}; vec {"0xdef"}}. Passing multiple text values in a nested vector corresponds to the "or" operation within the topics.

@rvanasa rvanasa merged commit 9a04b36 into main Feb 5, 2024
3 checks passed
@rvanasa rvanasa deleted the vec-topics branch February 5, 2024 16:33
gitlab-dfinity pushed a commit to dfinity/ic that referenced this pull request Feb 7, 2024
refactor(evmrpc): misc.

* Uses `Vec<Vec<FixedSizeData>>` to represent topics in place of `Vec<FixedSizeData>`.
* Refactors to support custom RPC services (such as L2s) in the Candid-RPC methods.

Prerequisite for [#158](dfinity/evm-rpc-canister#158) and [#165](dfinity/evm-rpc-canister#165). 

Merges into the `evm-rpc-canister` branch. 

See merge request dfinity-lab/public/ic!17485
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