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

Add a bool field to allow fractional fee to be charged to sender or receiver. #82

Closed

Conversation

ljianghedera
Copy link
Contributor

Description:

This PR adds one bool field to allow the user to decide the fractional custom fee to be charged to sender or receiver.

Related issue(s):

Fixes #81

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

anighanta
anighanta previously approved these changes Aug 4, 2021
qnswirlds
qnswirlds previously approved these changes Aug 4, 2021
@@ -34,6 +34,7 @@ message FractionalFee {
Fraction fractional_amount = 1; // The fraction of the transferred units to assess as a fee
int64 minimum_amount = 2; // The minimum amount to assess
int64 maximum_amount = 3; // The maximum amount to assess (zero implies no maximum)
bool netOfTransfers = 4; // If true, assesses the fee to the sender, so the receiver gets the full amount from the token transfer list, and the sender is charged an additional fee; if false, the receiver does NOT get the full amount, but only what is left over after paying the fractional fee

Choose a reason for hiding this comment

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

snake_case, for consistency

Signed-off-by: ljianghedera <[email protected]>
@ljianghedera ljianghedera dismissed stale reviews from qnswirlds and anighanta via 67f5113 August 6, 2021 14:37
Copy link

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

@nathanklick nathanklick deleted the 081-D-fractional-fee-charged-to-sender-or-receiver branch November 16, 2022 07:56
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.

Allow fractional fees to be charged to _either_ sender or receiver
5 participants