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 new property nextExchangeRate in the receipt #2696

Open
agadzhalov opened this issue Dec 4, 2024 · 0 comments · May be fixed by #2727
Open

Add new property nextExchangeRate in the receipt #2696

agadzhalov opened this issue Dec 4, 2024 · 0 comments · May be fixed by #2727
Assignees
Milestone

Comments

@agadzhalov
Copy link
Contributor

agadzhalov commented Dec 4, 2024

Description

Currently when you get the receipt of a transaction there's a property exchangeRate, but if you look at exchange_rate.proto you will see there is an ExchangeSetRate that contains two properties - currentRate and nextRate which is not returned on receipt. The exchangeRate returned to the client from the SDK displays only currentRate instead of both.

Current receipt

TransactionReceipt {
  transactionId = [email protected],
  status = SUCCESS,
  exchangeRate = ExchangeRate {
    hbars = 1,
    cents = 12,
    expirationTime = 2100-01-01T00:00:00Z,
    exchangeRateInCents = 12.0
  },
  accountId = 0.0.5162,
  fileId = null,
  contractId = null,
  topicId = null,
  tokenId = null,
  topicSequenceNumber = null,
  topicRunningHash = null,
  totalSupply = 0,
  scheduleId = null,
  scheduledTransactionId = null,
  serials = [],
  duplicates = [],
  children = []
}

Solution

Add new property nextExchangeRate to the receipt

Fix

TransactionReceipt.js#239

@agadzhalov agadzhalov changed the title Add new propertynextExchangeRate in the receipt Add new property nextExchangeRate in the receipt Dec 4, 2024
@agadzhalov agadzhalov added this to the v2.56.0 milestone Dec 10, 2024
@ivaylonikolov7 ivaylonikolov7 linked a pull request Dec 17, 2024 that will close this issue
2 tasks
@ivaylonikolov7 ivaylonikolov7 self-assigned this Dec 18, 2024
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 a pull request may close this issue.

2 participants