Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Zero refund check for FungibleAdapter #6506

Conversation

Dinonard
Copy link
Contributor

FungibleAdapter will now check if the refund amount is zero before calling deposit & emitting an event.

Fixes #6469.

@Dinonard Dinonard requested a review from a team as a code owner November 15, 2024 18:57
@franciscoaguirre franciscoaguirre added T2-pallets This PR/Issue is related to a particular pallet. A4-needs-backport Pull request must be backported to all maintained releases. labels Nov 15, 2024
@franciscoaguirre
Copy link
Contributor

/cmd prdoc

@franciscoaguirre
Copy link
Contributor

@Dinonard Please edit the prdoc, bump should be patch, audience "Runtime User"

@Dinonard
Copy link
Contributor Author

Forgot about PR doc, my bad.

Done now.

@Dinonard
Copy link
Contributor Author

Can this be merged via the bot?

@Dinonard
Copy link
Contributor Author

@franciscoaguirre could you please merge this?

@franciscoaguirre franciscoaguirre added this pull request to the merge queue Nov 26, 2024
Merged via the queue into paritytech:master with commit f520adb Nov 26, 2024
195 of 200 checks passed
@paritytech-cmd-bot-polkadot-sdk

Created backport PR for stable2407:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-6506-to-stable2407
git worktree add --checkout .worktree/backport-6506-to-stable2407 backport-6506-to-stable2407
cd .worktree/backport-6506-to-stable2407
git reset --hard HEAD^
git cherry-pick -x f520adb0aacd51247ed548c7db9bef874c2cca9e
git push --force-with-lease

@paritytech-cmd-bot-polkadot-sdk

Created backport PR for stable2409:

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin backport-6506-to-stable2409
git worktree add --checkout .worktree/backport-6506-to-stable2409 backport-6506-to-stable2409
cd .worktree/backport-6506-to-stable2409
git reset --hard HEAD^
git cherry-pick -x f520adb0aacd51247ed548c7db9bef874c2cca9e
git push --force-with-lease

EgorPopelyaev added a commit that referenced this pull request Dec 4, 2024
Backport #6506 into `stable2407` from Dinonard.

See the
[documentation](https://github.com/paritytech/polkadot-sdk/blob/master/docs/BACKPORT.md)
on how to use this bot.

<!--
  # To be used by other automation, do not modify:
  original-pr-number: #${pull_number}
-->

Co-authored-by: Dino Pačandi <3002868+Dinonard@users.noreply.github.com>
Co-authored-by: Egor_P <egor@parity.io>
EgorPopelyaev added a commit that referenced this pull request Dec 11, 2024
Backport #6506 into `stable2409` from Dinonard.

See the
[documentation](https://github.com/paritytech/polkadot-sdk/blob/master/docs/BACKPORT.md)
on how to use this bot.

<!--
  # To be used by other automation, do not modify:
  original-pr-number: #${pull_number}
-->

---------

Co-authored-by: Dino Pačandi <3002868+Dinonard@users.noreply.github.com>
Co-authored-by: EgorPopelyaev <egor@parity.io>
Krayt78 pushed a commit to Krayt78/polkadot-sdk that referenced this pull request Dec 18, 2024
`FungibleAdapter` will now check if the _refund amount_ is zero before
calling deposit & emitting an event.

Fixes paritytech#6469.

---------

Co-authored-by: GitHub Action <action@github.com>
dudo50 pushed a commit to paraspell-research/polkadot-sdk that referenced this pull request Jan 4, 2025
`FungibleAdapter` will now check if the _refund amount_ is zero before
calling deposit & emitting an event.

Fixes paritytech#6469.

---------

Co-authored-by: GitHub Action <action@github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
A4-needs-backport Pull request must be backported to all maintained releases. T2-pallets This PR/Issue is related to a particular pallet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

transfer_allow_death - emitting an empty Deposit event
5 participants