Fixbug on ItxnField opcode #550

vuvoth · 2022-01-13T09:55:59Z

Proposed Changes

Allow random address when creating asset and re-config when using the Inner transaction.
Teal ItxnField opcode understands address 32 bytes so add tests for this case.

robert-zaremba

Let's add more tests.

packages/runtime/src/interpreter/opcode.ts

packages/runtime/src/lib/itxn.ts

packages/runtime/test/src/interpreter/inner-transaction.ts

ratik21

Need to add test(s) before merging

CHANGELOG.md

packages/runtime/src/interpreter/opcode.ts

packages/runtime/src/lib/itxn.ts

packages/runtime/src/interpreter/opcode.ts

packages/runtime/test/src/interpreter/inner-transaction.ts

packages/runtime/src/lib/itxn.ts

vuvoth added 2 commits January 13, 2022 16:41

modify ITxnField accpect a random account address

891f4cb

update CHANGELOG.md

239f6dd

vuvoth requested review from robert-zaremba and ratik21 January 13, 2022 09:55

robert-zaremba reviewed Jan 13, 2022

View reviewed changes

fix review

4d70a8c

ratik21 reviewed Jan 13, 2022

View reviewed changes

vuvoth added 4 commits January 14, 2022 17:58

fix review

608e03c

set asset config account by random address

7aa5166

merge with develop

75b9f62

ignore check address exist in runtime when Tx is acfg

ddfbdde

vuvoth requested a review from ratik21 January 17, 2022 15:54

ratik21 approved these changes Jan 17, 2022

View reviewed changes

packages/runtime/src/lib/itxn.ts Outdated Show resolved Hide resolved

rename variable

b13abf1

vuvoth merged commit 8545f66 into develop Jan 18, 2022

vuvoth deleted the fixbug-in-ITxnField branch January 18, 2022 04:00