Mario
Mario
I understand, it also adds a bit more friction to developers coming from somewhere else or trying to re-use existing code from other projects that rely on ethers. Its one...
> I understand, it also adds a bit more friction to developers coming from somewhere else or trying to re-use existing code from other projects that rely on ethers. Its...
Due an update of the SDK I needed to update the transaction sending, its becoming more complex now and a single function is more needed now I believe. The current...
@Valazan Can you share an example on the improvement? I believe the optimal experience is a variation of this flow: ```ts /* setup env */ const thorClient = ThorClient.fromUrl('https://testnet.vechain.org'); const...
I browsed through the code and do you suggest the use of`executeMultipleClausesTransaction`? It is great and very close, but I believe it requires two adjustments to be most useful: 1....
@Valazan that's great to know! thank you 🙏 With the function `filterEventLogs` still differs from the other function that support fragments, I believe it should be in there too. With...
@Valazan @nwbrettski May I suggest to also decode the data before returning it? The same is already happening on calling contract functions, the returned data is decoded. Asking the user...
@Valazan thank you for making the improvement 🙌 I have a question regarding compatibility. # Syntax As far as I understand you changed it from: ```ts const filteredLogs = await...
> Can we also add ABI support, so we're fully compatible with current vechain signers? I was somehow expecting this to happen already automatically in the background for the `clauseBuilder`...
@darrenvechain This looks like a good spot to add it and having it enabled for all calls by default: https://github.com/vechain/vechain-sdk-js/blob/b27bb34a6b5e6af44daa8362c79c07e06d6cd091/packages/core/src/clause/clause.ts#L46-L57 Is it possible that the [`TransactionClause`](https://tsdocs.dev/docs/@vechain/sdk-network/1.0.0-beta.3/interfaces/_internal_.TransactionClause.html) is missing the [`TxMessage`](https://github.com/vechain/connex/blob/master/packages/types/vendor.d.ts#L64-L69)...