-
Notifications
You must be signed in to change notification settings - Fork 79
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
chore: docs: add user contract api docs #1210
base: release/v10
Are you sure you want to change the base?
Conversation
c84ae28
to
ce65ecb
Compare
This work is heavily based of @longfeiWan9's work at https://github.com/longfeiWan9/BuiltIn-Actor-API
ce65ecb
to
47f0cea
Compare
@jennijuju I have no access to directly push to your branch in this repo. Will you grant me access? |
Done |
Update this PR based on the checklist. |
for evm actor it doesn't have exported method tho! |
@jennijuju added read-only flags according to the nature of built-in actor methods as well as read-only flags set in filecoin.solidity. I think this is ready, maybe someone can review and sign off to merge ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rubber stamp. I haven't reviewed all the docs for accuracy. In this case we're deciding that something is better than nothing.
@jennijuju we want this merged into master right? |
This work is heavily based of @longfeiWan9's work at https://github.com/longfeiWan9/BuiltIn-Actor-API
Todos before merge: