Upgrade to ton v13 #182
Upgrade to ton v13 #182
Conversation
| } | ||
| }); | ||
| } | ||
| return parsedItems; | ||
| } | ||
|
|
||
| export async function makeGetCall<T>( |
There was a problem hiding this comment.
do we still need makeGetCall anywhere in this codebase?
There was a problem hiding this comment.
Yes, for calling get methods on submitted contracts (Getters component)
src/lib/useContractAddress.ts
Outdated
| const verifiedAddressStr = verifiedAddress?.toString() ?? null; | ||
| const verifiedAddresssHex = verifiedAddress?.toString() ?? null; |
There was a problem hiding this comment.
this looks suspicious - if we wanted one friendly representation (EQDerEP...) and one hex (0:deac43d3221d0e...) we need different calls
There was a problem hiding this comment.
Right, toString is now toRawString in ton v13. Fixed it.
src/lib/makeGetCall.ts
Outdated
| return ["tvm.Slice", p.toBoc({ idx: false }).toString("base64")]; | ||
| } else if (p instanceof BN) { | ||
| return ["num", p.toString(10)]; | ||
| paramsTuple.writeCell(p); |
There was a problem hiding this comment.
this should be writeSlice
src/lib/makeGetCall.ts
Outdated
| } | ||
|
|
||
| throw new Error("unknown type!"); |
There was a problem hiding this comment.
this always gets called
There was a problem hiding this comment.
still needs addressing after latest fixes (e.g. put in else clause)
| case "list": | ||
| if (val.elements.length === 0) { | ||
| return null; | ||
| parsedItems.push((item as TupleItemCell).cell); |
There was a problem hiding this comment.
missing break in these switches
No description provided.