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

doc: Dynamic Cross-Contract Calling #270

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

0xf333
Copy link
Contributor

@0xf333 0xf333 commented Aug 27, 2023

Description

  • Updated ink! code snippets to conform to v4.
  • Introduced working solidity equivalent to make the comparison easier to grasp.
  • Removed comment regarding v3 limitations

@0xf333 0xf333 force-pushed the 0xf333/submit_generic_transaction branch from da791e4 to 065f190 Compare August 28, 2023 12:57
@0xf333 0xf333 requested a review from SkymanOne August 28, 2023 13:04
@0xf333
Copy link
Contributor Author

0xf333 commented Aug 28, 2023

@SkymanOne

This should probably be fixed in a separated pull request since it's not part of this PR scope.

Check Markdown links / markdown-link-check (pull_request) 

Let me know what you think.

Description
============
- Updated ink! code snippets to conform to v4.
- Introduced working solidity equivalent to
  make the comparison easier to grasp.
- Removed comment regarding v3 limitations
@0xf333 0xf333 force-pushed the 0xf333/submit_generic_transaction branch from 065f190 to a71a3a5 Compare August 31, 2023 07:00
@0xf333
Copy link
Contributor Author

0xf333 commented Sep 4, 2023

@SkymanOne

This should probably be fixed in a separated pull request since it's not part of this PR scope.

Check Markdown links / markdown-link-check (pull_request) 

Let me know what you think.

@SkymanOne
markdown-link-check failure has been fixed in #273 #272 #271

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.

2 participants