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.

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

Implement standard token receiver hook in Account and Multisig actors #538

Closed
anorth opened this issue Aug 12, 2022 · 2 comments
Closed
Assignees

Comments

@anorth
Copy link
Member

anorth commented Aug 12, 2022

Implement a token receiver hook as described by filecoin-project/FIPs#407. For these actors, the hook should always accept a transfer.

See also #525.

@anorth anorth added the P1 label Aug 12, 2022
@anorth anorth moved this to Todo / In Scope in Network nv17 Aug 12, 2022
@ZenGround0
Copy link
Contributor

One thing to work out

One design question is whether the token receiver hook should receive addresses as Address type or ActorID type.

I lean towards using actor id

@anorth anorth self-assigned this Aug 16, 2022
@anorth anorth moved this from Todo / In Scope to In Progress in Network nv17 Aug 16, 2022
@anorth anorth moved this from In Progress to In Review in Network nv17 Aug 18, 2022
@anorth
Copy link
Member Author

anorth commented Aug 18, 2022

Implemented in #555

@anorth anorth closed this as completed Aug 18, 2022
Repository owner moved this from In Review to Done in Network nv17 Aug 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Status: Done
Development

No branches or pull requests

3 participants