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

wallet: feature: transfer amount with fee included [RELEASE] #8945

Merged

Conversation

jeffro256
Copy link
Contributor

@vtnerd
Copy link
Contributor

vtnerd commented Jul 12, 2023

Does this feature really need to be ported to the stable release branch?

@jeffro256
Copy link
Contributor Author

Why wouldn't it be? It's a useful feature. Are you saying it hasn't been reviewed enough? (True, it hasn't, but then we just don't approve it yet)

@vtnerd
Copy link
Contributor

vtnerd commented Jul 13, 2023

In many projects, new features are not ported to an existing release branch to keep it stable. So I'm asking whether its worth breaking the possible stability of the current release branch for this feature.

@woodser
Copy link
Contributor

woodser commented Oct 8, 2023

Having it on a stable release branch allows me to pull and test the feature against my library test suites and even release experimental features ahead of monero-project, whereas I'm not able to do that against master due to incompatibilities and the possibility of unreleased bugs.

@jeffro256 jeffro256 force-pushed the subtract_fee_from_outputs_18 branch 2 times, most recently from 6987981 to cb2df5d Compare November 10, 2023 21:18
@jeffro256 jeffro256 marked this pull request as draft November 10, 2023 21:19
@jeffro256
Copy link
Contributor Author

Putting this PR in draft mode b/c I don't want it to be merged yet

@jeffro256 jeffro256 marked this pull request as ready for review February 20, 2024 19:26
To transfer ~5 XMR to an address such that your balance drops by exactly 5 XMR, provide a `subtractfeefrom` flag to the `transfer` command. For example:

    transfer 76bDHojqFYiFCCYYtzTveJ8oFtmpNp3X1TgV2oKP7rHmZyFK1RvyE4r8vsJzf7SyNohMnbKT9wbcD3XUTgsZLX8LU5JBCfm 5 subtractfeefrom=all

If my walet balance was exactly 30 XMR before this transaction, it will be exactly 25 XMR afterwards and the destination address will receive slightly
less than 5 XMR. You can manually select which destinations fund the transaction fee and which ones do not by providing the destination index.
For example:

    transfer 75sr8AAr... 3 74M7W4eg... 4 7AbWqDZ6... 5 subtractfeefrom=0,2

This will drop your balance by exactly 12 XMR including fees and will spread the fee cost proportionally (3:5 ratio) over destinations with addresses
`75sr8AAr...` and `7AbWqDZ6...`, respectively.

Disclaimer: This feature was paid for by @LocalMonero.
@luigi1111 luigi1111 merged commit f9a7f2a into monero-project:release-v0.18 Feb 24, 2024
17 checks passed
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.

None yet

5 participants