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

feat: document multicall 3 #2291

Merged
merged 19 commits into from
Aug 6, 2024
Merged

Conversation

xBalbinus
Copy link
Contributor

No description provided.

@snissn
Copy link
Contributor

snissn commented Jul 25, 2024

  1. CI fail issue
  2. The Contract address section table does not look formatted right

@xBalbinus
Copy link
Contributor Author

CI & table Fixed @snissn - thanks for the heads up.

smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Show resolved Hide resolved
smart-contracts/advanced/multicall.md Outdated Show resolved Hide resolved
smart-contracts/advanced/multicall.md Show resolved Hide resolved
@longfeiWan9
Copy link
Member

@xBalbinus Thanks for considering my comments. The shortened version of explain CALL and DELEGATECALL looks great. I only have two more small suggestions. Otherwise, it looks great to me.

@xBalbinus
Copy link
Contributor Author

Have merged your suggestions @longfeiWan9 ! Please merge this into main, I do not have permissions.

@longfeiWan9 longfeiWan9 merged commit 6232d5a into filecoin-project:main Aug 6, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants