-
Notifications
You must be signed in to change notification settings - Fork 12
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
fix: revert restricted balances #4237
Conversation
PRO-971 Restrictions are lost if a redemption reverts
How to reproduceCan the bug be reproduced? How? Ideally, provide a bouncer test or unit test that demonstrated the problem. Redeem to a restricted address and let it expire. SymptomsHow does the bug manifest? If it's not obvious, what behaviour did we expect? The funds are credited back to the account but they are now unrestricted. |
FYI @acdibble (thanks for raising this). |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #4237 +/- ##
======================================
- Coverage 72% 71% -0%
======================================
Files 384 384
Lines 63015 63180 +165
Branches 63015 63180 +165
======================================
+ Hits 45074 45122 +48
- Misses 15609 15723 +114
- Partials 2332 2335 +3 ☔ View full report in Codecov by Sentry. |
Co-authored-by: dandanlen <3168260+dandanlen@users.noreply.github.com> Co-authored-by: Alastair Holmes <42404303+AlastairHolmes@users.noreply.github.com> Co-authored-by: Albert Llimos <53186777+albert-llimos@users.noreply.github.com> Co-authored-by: Martin Rieke <121793148+martin-chainflip@users.noreply.github.com> Co-authored-by: Maxim Shishmarev <msgmaxim@gmail.com> Co-authored-by: Marcello <marcello@chainflip.io> Co-authored-by: Roy Yang <roy@chainflip.io> Co-authored-by: kylezs <kyle@chainflip.io> Co-authored-by: Jamie Ford <jamie@chainflip.io> fix naming of session keys (#4242) fix: revert restricted balances (#4237) fix: add missing spans in multisig logs (#4239) fix cargo features (#4249) fix: remove bound addresses on account deletion (#4244) fix: remove existential deposit (#4243) fix: egress id race condition (#4235) fix: remove unwrap when getting tx receipt (#4231) fix: protect against double witnessing after safe mode (#4254) fix: runtime upgrade utils and migrations (#4258)
Pull Request
Closes: PRO-971
Checklist
Please conduct a thorough self-review before opening the PR.
Summary
This fixes two issues: