Skip to content

Conversation

@mohd-kashif
Copy link
Contributor

@mohd-kashif mohd-kashif commented Nov 19, 2025

  • removed the old test cases file as they were insignificant
  • refactored the transaction builder as related inheritance
  • added exportInCTxBuilder and its test cases only, will add other txn builder in later PRs

@mohd-kashif mohd-kashif marked this pull request as ready for review November 25, 2025 08:06
@mohd-kashif mohd-kashif requested review from a team as code owners November 25, 2025 08:06
@mohd-kashif mohd-kashif changed the title WIN-7770-3 WIN-7770-3 | flrp export C to P transaction builder Nov 25, 2025
@mohd-kashif
Copy link
Contributor Author

there are some refactoring and Todos which i will be picking in the subsequent PRs for import transaction builders
will squash the commits post review

@mohd-kashif mohd-kashif force-pushed the WIN-7770-3 branch 2 times, most recently from 3171fb0 to ed8cbc9 Compare November 26, 2025 08:31
Copy link
Contributor

@parasgarg-bitgo parasgarg-bitgo left a comment

Choose a reason for hiding this comment

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

Please also consider breaking the PR into multiple smaller PRs. I'd suggest breaking it down in 3 PRs

  1. Remove irrelevant code
  2. Add transaction base and concrete implementations
  3. Add transaction builder and factory

@mohd-kashif mohd-kashif merged commit 2962195 into master Nov 27, 2025
14 checks passed
@mohd-kashif mohd-kashif deleted the WIN-7770-3 branch November 27, 2025 13:58
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.

4 participants