Skip to content

Conversation

rukmini-basu-da
Copy link
Contributor

@rukmini-basu-da rukmini-basu-da commented Oct 15, 2025

Closes #492

@rukmini-basu-da rukmini-basu-da marked this pull request as ready for review October 15, 2025 16:59
@rukmini-basu-da rukmini-basu-da requested review from a team as code owners October 15, 2025 16:59
contractId: proxyCid,
choice: 'DelegateProxy_TransferInstruction_Reject',
choiceArgument: choiceArgs,
},
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rukmini-basu-da @PHOL-DA : any particular reason that you are not leveraging the codegen's types for constructing this command?

Copy link
Contributor Author

@rukmini-basu-da rukmini-basu-da Oct 15, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we have this issue that will allow us to use the codegen's types

Copy link
Contributor

@PHOL-DA PHOL-DA left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks very nice 😄 might have gone a little overboard with nit picking

@rukmini-basu-da rukmini-basu-da force-pushed the rukmini/rewards-for-deposits branch from 2db3736 to 044aecc Compare October 16, 2025 12:27
rukmini-basu-da and others added 11 commits October 16, 2025 09:58
…erInstruction_Withdraw

Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
…deposits.ts

Co-authored-by: Phillip Olesen <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Co-authored-by: Phillip Olesen <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
@rukmini-basu-da rukmini-basu-da force-pushed the rukmini/rewards-for-deposits branch from d582be2 to 874694b Compare October 16, 2025 13:58
@rukmini-basu-da rukmini-basu-da marked this pull request as draft October 16, 2025 14:01
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
Signed-off-by: rukmini-basu-da <[email protected]>
@rukmini-basu-da rukmini-basu-da marked this pull request as ready for review October 16, 2025 15:23
Signed-off-by: rukmini-basu-da <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Wallet SDK] Feature app marker

3 participants