abi-decoder icon indicating copy to clipboard operation
abi-decoder copied to clipboard

handle multiple events with the same normalized form.

Open efiShtain opened this issue 5 years ago • 3 comments

In case that multiple Abis are loaded, and there is an event in multiple forms (with/without indexed arguments). This commit tries to handle all cases and returns the first one that succeed or undefined otherwise.

In previous version, an exception is thrown although a valid ABI is supplied since the last loaded ABI overrides the previous ones

efiShtain avatar Sep 30 '19 14:09 efiShtain

@efiShtain this fix seems much needed, not sure why it does not get merged

have you figured out another way since you opened your PR?

decentraliser avatar Apr 15 '21 11:04 decentraliser

@decentraliser Sorry, didn't follow up since then not sure why it did not get merge

efiShtain avatar Apr 19 '21 17:04 efiShtain

hi @efiShtain , I published a package on npm, logs-decoder, that is a fork of this one. I integrated the changes you propose here. https://github.com/decentraliser/logs-decoder/pull/4/files

decentraliser avatar Apr 26 '21 13:04 decentraliser