Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

use can_dec_provider to check when transfer allow death #9411

Merged
merged 2 commits into from
Jul 27, 2021
Merged

use can_dec_provider to check when transfer allow death #9411

merged 2 commits into from
Jul 27, 2021

Conversation

wangjj9219
Copy link
Contributor

related to #9375

@cla-bot-2021
Copy link

cla-bot-2021 bot commented Jul 22, 2021

User @wangjj9219, please sign the CLA here.

shawntabrizi
shawntabrizi approved these changes Jul 24, 2021
View reviewed changes
Copy link
Member

@shawntabrizi shawntabrizi left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good to me, thanks for catching this

@shawntabrizi shawntabrizi added B7-runtimenoteworthy C3-medium PR touches the given topic and has a medium impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited labels Jul 24, 2021
apopiak
apopiak approved these changes Jul 27, 2021
View reviewed changes
Copy link
Contributor

@apopiak apopiak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gavofyork gavofyork merged commit 1b4b05d into paritytech:master Jul 27, 2021
@viniul viniul added D1-audited 👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited and removed D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited labels Aug 4, 2021
@shaunxw shaunxw mentioned this pull request Aug 9, 2021
Closed
@github-actions github-actions bot mentioned this pull request Aug 24, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@apopiak apopiak apopiak approved these changes

@shawntabrizi shawntabrizi shawntabrizi approved these changes

@gavofyork gavofyork gavofyork approved these changes

Assignees
No one assigned
Labels
C3-medium PR touches the given topic and has a medium impact on builders. D1-audited 👍 PR contains changes to fund-managing logic that has been properly reviewed and externally audited
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@wangjj9219 @gavofyork @shawntabrizi @apopiak @viniul