Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Utility] Account Logic - Send and Receive Funds - Add Tx Fees to Mempool #326

Closed
8 tasks
jessicadaugherty opened this issue Oct 31, 2022 · 2 comments
Closed
8 tasks

Comments

@jessicadaugherty
Copy link
Contributor

Objective

Add tx fees from all send txs to mempool. Blocked by #325

Origin Document

The foundations for sending transactions were built out here but they do not include adding tx fees to the mempool.

Goals

  • Add more production ready functionality for sending E2E txs in localnet and devnet

Deliverable

  • Move tx fees with funds to mempool
  • Store tx fees with funds in mempool

Non-goals / Non-deliverables

  • Implementing "unhappy path" for save points

General issue deliverables

  • Update the appropriate CHANGELOG
  • Update any relevant READMEs (local and/or global)
  • Update any relevant global documentation & references
  • If applicable, update the source code tree explanation
  • If applicable, add or update a state, sequence or flowchart diagram using mermaid

[Optional] Testing Methodology

  • Task specific tests: make ...
  • All tests: make test_all
  • LocalNet: verify a LocalNet is still functioning correctly by following the instructions at docs/development/README.md

Creator: @jessicadaugherty

@Olshansk
Copy link
Member

@jessicadaugherty Given the updated description of #325, I believe we can close this out as a duplicate. Wdyt?

@jessicadaugherty
Copy link
Contributor Author

@Olshansk aligned

@jessicadaugherty jessicadaugherty closed this as not planned Won't fix, can't repro, duplicate, stale Oct 31, 2022
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

No branches or pull requests

2 participants