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

Custom fees support design #2173

Merged
merged 13 commits into from
Jul 2, 2021
Merged

Custom fees support design #2173

merged 13 commits into from
Jul 2, 2021

Conversation

xin-hedera
Copy link
Collaborator

@xin-hedera xin-hedera commented Jun 24, 2021

Detailed description:

This is just for the design review.

Which issue(s) this PR fixes:
Relates to #2166

Special notes for your reviewer:

This is the hedera-protobufs custom fees PR (hashgraph/hedera-protobufs#42).

Update: the protobufs PR is merged and made into Hedera protobufs v0.16.0-alpha.1 release.

Checklist

  • Documentation added
  • Tests updated

Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
@xin-hedera xin-hedera added enhancement Type: New feature P1 parser Area: File parsing hts Hedera Token Service labels Jun 24, 2021
@xin-hedera xin-hedera added this to the Mirror 0.37.0 milestone Jun 24, 2021
@xin-hedera xin-hedera requested a review from a team June 24, 2021 16:16
@xin-hedera xin-hedera self-assigned this Jun 24, 2021
Signed-off-by: Xin Li <xin.li@hedera.com>
@codecov
Copy link

codecov bot commented Jun 24, 2021

Codecov Report

❗ No coverage uploaded for pull request base (main@baee393). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #2173   +/-   ##
=======================================
  Coverage        ?   81.69%           
  Complexity      ?     2134           
=======================================
  Files           ?      414           
  Lines           ?    11150           
  Branches        ?      945           
=======================================
  Hits            ?     9109           
  Misses          ?     1732           
  Partials        ?      309           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update baee393...c0f4b02. Read the comment docs.

Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
@xin-hedera xin-hedera changed the title Custom fees support importer Custom fees support design Jun 24, 2021
@xin-hedera xin-hedera marked this pull request as ready for review June 24, 2021 20:33
@xin-hedera xin-hedera added design and removed parser Area: File parsing labels Jun 24, 2021
Signed-off-by: Xin Li <xin.li@hedera.com>
docs/design/custom-fees.md Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Xin Li <xin.li@hedera.com>
amount bigint not null,
amount_denominator bigint,
collector_account_id bigint,
created_timestamp bigint not null,
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

renamed it from consensus_timestamp, created better describes the field and also in the token info json response, created_timestamp is easier to understand and more distinguishable when it's mixed with the token's created and modified timestamp.

Signed-off-by: Xin Li <xin.li@hedera.com>
@xin-hedera xin-hedera marked this pull request as draft June 30, 2021 17:01
@xin-hedera
Copy link
Collaborator Author

moved it back to draft since there will be incompatible changes in protobufs for HIP-18.

Signed-off-by: Xin Li <xin.li@hedera.com>
@xin-hedera xin-hedera marked this pull request as ready for review July 1, 2021 14:34
Signed-off-by: Xin Li <xin.li@hedera.com>
docs/design/custom-fees.md Show resolved Hide resolved
docs/design/custom-fees.md Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
docs/design/custom-fees.md Show resolved Hide resolved
docs/design/custom-fees.md Outdated Show resolved Hide resolved
Signed-off-by: Xin Li <xin.li@hedera.com>
collector_account_id bigint,
created_timestamp bigint not null,
denominating_token_id bigint,
maximum_amount bigint,
Copy link
Collaborator Author

@xin-hedera xin-hedera Jul 2, 2021

Choose a reason for hiding this comment

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

removed has_custom_fee

@xin-hedera xin-hedera requested a review from Nana-EC July 2, 2021 21:37
@sonarcloud
Copy link

sonarcloud bot commented Jul 2, 2021

Copy link
Contributor

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

LGTM

@xin-hedera xin-hedera merged commit 8730c07 into main Jul 2, 2021
@xin-hedera xin-hedera deleted the custom_fees branch July 2, 2021 22:01
ijungmann pushed a commit that referenced this pull request Jul 16, 2021
- Add HTS custom fees design doc

Signed-off-by: Xin Li <xin.li@hedera.com>
Signed-off-by: Ian Jungmann <ian.jungmann@hedera.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design enhancement Type: New feature hts Hedera Token Service P1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants