Skip to content

feat: update TransactionEffects#580

Open
Alex6323 wants to merge 12 commits intomove-core-sdkfrom
feat/update-transaction-effects
Open

feat: update TransactionEffects#580
Alex6323 wants to merge 12 commits intomove-core-sdkfrom
feat/update-transaction-effects

Conversation

@Alex6323
Copy link
Contributor

@Alex6323 Alex6323 commented Feb 16, 2026

This PR adds the enum_dispatch convenience to the SDK for the monorepo by introducing TransactionEffectsAPI trait and implementing its methods.

Splits the TransactionEffectsApi trait, so that methods "for_testing" are separated.

@Alex6323 Alex6323 marked this pull request as ready for review February 17, 2026 11:00
@Alex6323 Alex6323 requested a review from a team as a code owner February 17, 2026 11:00
@Alex6323 Alex6323 force-pushed the feat/update-transaction-effects branch from 5e05157 to bb9b63e Compare February 18, 2026 15:06
assert!(effects.is_ok(), "Execution failed. Effects: {effects:?}");
// check that it succeeded
let status = effects.unwrap();
let effects = effects.unwrap().into_v1();
Copy link
Contributor

Choose a reason for hiding this comment

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

We should probably have a match for places like this so we'll know to update them if we add new versions

@Alex6323 Alex6323 force-pushed the feat/update-transaction-effects branch from 6cc65b9 to 8579bce Compare March 5, 2026 18:28
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.

3 participants