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.

Sign up for GitHub

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

Jump to bottom

[venus] add check for datacap / datacap actor增加一个check？ #5514

Closed
1 task
Tracked by #5527
Fatman13 opened this issue Nov 28, 2022 · 0 comments · Fixed by #5542
Closed
1 task
Tracked by #5527

[venus] add check for datacap / datacap actor增加一个check？ #5514

Fatman13 opened this issue Nov 28, 2022 · 0 comments · Fixed by #5542
Assignees
@simlecode
Labels
CU-chain-service Venus chain service related issues P3 Low - not important right now

Comments

@Fatman13
Copy link
Contributor

Thanks @frrist for pointing this out. This call should never error anyway, as long as the datacap actor keeps a correct state, but adding this check for safety.

关联：
@Fatman13 Fatman13 added CU-chain-service Venus chain service related issues C-triage Category: Need triage labels Nov 28, 2022
@Fatman13 Fatman13 changed the title [venus] datacap actor增加一个check？ [venus] add check for datacap / datacap actor增加一个check？ Nov 28, 2022
@simlecode simlecode removed their assignment Nov 28, 2022
@simlecode simlecode added P3 Low - not important right now and removed C-triage Category: Need triage labels Nov 28, 2022
@diwufeiwen diwufeiwen mentioned this issue Dec 5, 2022
Closed
42 tasks
@simlecode simlecode self-assigned this Dec 6, 2022
@simlecode simlecode moved this to In Progress in Venus Project Dec 6, 2022
@simlecode simlecode mentioned this issue Dec 6, 2022
Merged
6 tasks
Repository owner moved this from In Progress to Done in Venus Project Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simlecode simlecode

Labels
CU-chain-service Venus chain service related issues P3 Low - not important right now
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

chore: update deps filecoin-project/venus
2 participants
@Fatman13 @simlecode