-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Update index.md #3444
Update index.md #3444
Conversation
Edited for punctuation and a broken link.
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.
@jbgwu Thanks for posting =) Left a question about the link you swapped out, otherwise this looks good 👍🏻
In this article we will analyze [Ryuya Nakamura's ERC-721 contract](https://github.com/vyperlang/vyper/blob/master/examples/tokens/ERC721.vy). | ||
In this article we will analyze [Ryuya Nakamura's ERC-721 contract](https://github.com/vyperlang/vyper/blob/master/vyper/builtin_interfaces/ERC721.py). |
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.
Little confused, the original link is working for me and appears to be the contract being referenced. Could we revert this? Or am I missing something?
Hey @jbgwu ✌️ Thanks for the PR. Agree with Paul that the original reference to the vyper file was the correct one so I've reverted that (feel free to comment if you think we've made a mistake). Appreciate the improvements you've made. Be sure to join the discord if you are interested in contributing further to the project or have any questions for the team. And remember to claim your POAP also 🥳!. |
@all-contributors please add @jbgwu for docs |
I've put up a pull request to add @jbgwu! 🎉 |
Edited for punctuation and a broken link.
Description
Related Issue