Jamie
Jamie
Is this closed??
I wrote down [how to perform rosetta-klaytn's check:data test](https://github.com/klaytn/rosetta-klaytn/wiki/How-to-run-check:data-testing-for-data-consistency). However, rosetta-klaytn v1.0.1 must be released to work according to this guide, so after v1.0.1 version is released, i will put...
@aidan-kwon I checked the [testing guide](https://github.com/klaytn/rosetta-klaytn/wiki/How-to-run-check:data-testing-for-data-consistency) with rosetta-klaytn v1.0.1 (latest version). cc @kjhman21
I think `caver.rpc.klay.createAccessList` should receive a transaction instance. When i work on this, i will discuss about this with you. Thank you for reporting.
I guess you are working on web, and buffer dependency causes an error. I don't think we need to add buffer dependency separate, because we are using Node.js native module....
I'm testing it with your branch and it looks like there are no issues. But can you add one test code to make sure this works as intended?
I think we should first discuss whether this change is a necessary change. In this case, the contract deployment is failed in EVM, but the transaction is processed and included...
Okay i understand your point. I took a look more logic, and in my opinion, `if (!receipt.status && receipt.txError)` logic should be modified. So i will add suggestion for this.
https://kips.klaytn.com/KIPs/kip-34
@Byeong-jun-Kim You can use caver-js v1.9.1-rc.1. Thanks