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

Remove trait method TransactionCompat::otterscan_api_truncate_input #11163

Closed
emhane opened this issue Sep 24, 2024 · 8 comments · Fixed by #12236
Closed

Remove trait method TransactionCompat::otterscan_api_truncate_input #11163

emhane opened this issue Sep 24, 2024 · 8 comments · Fixed by #12236
Assignees
Labels
A-rpc Related to the RPC implementation C-debt Refactor of code section that is hard to understand or maintain D-good-first-issue Nice and easy! A great choice to get started S-blocked This cannot more forward until something else changes

Comments

@emhane
Copy link
Member

emhane commented Sep 24, 2024

Describe the feature

Remove

/// Truncates the input of a transaction to only the first 4 bytes.
// todo: remove in favour of using constructor on `TransactionResponse` or similar
// <https://github.com/alloy-rs/alloy/issues/1315>.
fn otterscan_api_truncate_input(tx: &mut Self::Transaction);

in favour of alloy-rs/alloy#1346

Additional context

No response

@emhane emhane added S-blocked This cannot more forward until something else changes C-debt Refactor of code section that is hard to understand or maintain A-rpc Related to the RPC implementation labels Sep 24, 2024
@emhane emhane added the D-good-first-issue Nice and easy! A great choice to get started label Sep 24, 2024
@emhane
Copy link
Member Author

emhane commented Sep 24, 2024

blocked by alloy-rs/alloy#1346

@nadtech-hub
Copy link
Contributor

I would like to take on this one

@nadtech-hub
Copy link
Contributor

It should be closed @emhane

@emhane
Copy link
Member Author

emhane commented Oct 30, 2024

by which pr @nadtech-hub ?

@nadtech-hub
Copy link
Contributor

nadtech-hub commented Oct 30, 2024

PR linked with issue that you had left reference to was closed. So rather removing this method it has to be refactored in some way. Do you have a suggestion? Issue need be edited as well in that case

@emhane
Copy link
Member Author

emhane commented Oct 30, 2024

I don't have any objections to alloy-rs/alloy#1354, will need @mattsse s input here

@nadtech-hub
Copy link
Contributor

nadtech-hub commented Oct 30, 2024

why do you need to truncate input. After reading otterscan docs I couldn't figure it out

@nadtech-hub
Copy link
Contributor

Just a reminder that tx's input may contains arbitrary data not only fn call data

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-rpc Related to the RPC implementation C-debt Refactor of code section that is hard to understand or maintain D-good-first-issue Nice and easy! A great choice to get started S-blocked This cannot more forward until something else changes
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants