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

feat: 🎸 Return balance info when applying incoming balance #30

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

prashantasdeveloper
Copy link

Description

Return balance info when applying incoming balance

Breaking Changes

On applying incoming balance(s) for an account, the method now returns details about the confidential asset, the amount being applied and balance after amount is deposited.

  • confidentialAccounts.applyIncomingBalance returns IncomingConfidentialAssetBalance
  • confidentialAccounts.applyIncomingBalances returns IncomingConfidentialAssetBalance[]

JIRA Link

DA-1264

Checklist

  • Updated the Readme.md (if required) ?

@prashantasdeveloper prashantasdeveloper requested a review from a team as a code owner July 26, 2024 12:56
BREAKING CHANGE: 🧨 On applying incoming balance(s) for an account,
the method now returns details about the confidential asset,
the amount being applied and balance after amount is deposited.
- `confidentialAccounts.applyIncomingBalance` returns
`IncomingConfidentialAssetBalance`
- `confidentialAccounts.applyIncomingBalances` returns
`IncomingConfidentialAssetBalance[]`
Copy link

sonarcloud bot commented Jul 26, 2024

@prashantasdeveloper
Copy link
Author

/fast-forward

@prashantasdeveloper prashantasdeveloper merged commit 81e5c1d into alpha Jul 30, 2024
11 checks passed
@prashantasdeveloper prashantasdeveloper deleted the feat/DA-1264 branch July 30, 2024 07:07
@polymath-eric
Copy link
Collaborator

🎉 This PR is included in version 2.0.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants