paritytech / substrate

Substrate: The platform for blockchain innovators
Apache License 2.0
8.39k stars 2.65k forks source link

Pallets: Assets `destroy_accounts` releases the deposit #14443

Closed muharem closed 1 year ago

muharem commented 1 year ago

Destroy accounts should release the existence deposits if any.

Currently, when accounts are being destroyed, deposits are not released. The dead_account function is executed forcefully, allowing the removal of the asset even if some accounts were not removed correctly. In the PR we release the deposits on the account destroy and do not force the dead_account function.

muharem commented 1 year ago

bot merge

paritytech-processbot[bot] commented 1 year ago

Waiting for commit status.

muharem commented 1 year ago

bot rebase

paritytech-processbot[bot] commented 1 year ago

Rebased

paritytech-processbot[bot] commented 1 year ago

Merge cancelled due to error. Error: Head SHA changed from 879ddb9a9c3c71d2785bc47f137d16aa55b9c078 to 702abcfb81cda26871ff8d3efe27d6981170d8da

muharem commented 1 year ago

bot merge