Refactor send-transaction to use NewTransaction

[mcintyre94]

This PR replaces the helpers for sending transactions to send NewTransaction objects, and to use the lifetime helpers.

We replace SendableTransaction with FullySignedTransaction, and mix in the lifetime constraints when required.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 29d3a4d

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[mcintyre94]

• Refactor send-transaction to use NewTransaction #2487 : 2 dependent PRs (#2488 , #2492 ) 👈
• add TransactionWithXLifetime types #2486
• Refactor transaction-confirmation to use new transaction lifetimes #2485
• Maintain transaction lifetime on transactions #2484
• Attach a typed lifetime constraint field when we compile a transaction #2481
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mcintyre94 and the rest of your teammates on Graphite

[mcintyre94]

This is now just cleaning up some stuff earlier in the stack, mostly in #2485 - I had to pull most of the code changes back to that branch

[steveluscher]

[mcintyre94]

Merge activity

• Apr 18, 2:34 AM EDT: @mcintyre94 started a stack merge that includes this pull request via Graphite.
• Apr 18, 2:35 AM EDT: Graphite rebased this pull request as part of a merge.
• Apr 18, 2:36 AM EDT: @mcintyre94 merged this pull request with Graphite.

[github-actions]

🎉 This PR is included in version 1.91.7 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[github-actions]

Because there has been no activity on this PR for 14 days since it was merged, it has been automatically locked. Please open a new issue if it requires a follow up.