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

Erc20 lesson update 19nov #268

Merged
merged 8 commits into from
Nov 23, 2023
Merged

Erc20 lesson update 19nov #268

merged 8 commits into from
Nov 23, 2023

Conversation

elPiablo
Copy link
Collaborator

@elPiablo elPiablo commented Nov 19, 2023

PR aims to get Create your first ERC-20 with Foundry lesson fully ready for production in 5 to 6 stages/commits

  • 1. OpenZeppelin have updated their releases to solc version 0.8.20 with breaking changes. Our first commit will:

update solidity pragma statements; redeploy contact; update terminal output code; renew Etherscan screenshots (public/assets/lessons) (see commits: 218e058 and 310cdae)

  • 2. add 1/4 and 2/4 of checkpoint questions (see commit: c2d7cc5)

  • 3. add 3/4 and 4/4 of checkpoint questions (see commit: 0b79dec )

  • 4. add final quiz (see commit: 21b61b4)

  • 5. add outro final paragraph for learners' next possible steps including other tracks and https://handbook.developerdao.com/ (see commit: )

- [ ] 6. Need to get info on POKT (Nodies and Grove) - there doesn't seem to be a shared public URL for an endpoint. And need to get in touch with Zerion to give us a little guide on what it is and how to set up. (see issue: #269 (comment)) we have an open issue for this, so we can ignore it in the context of this PR

  • extra: add SEO friendly title and description (see commit: 7773888)

Copy link

vercel bot commented Nov 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
academy ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 22, 2023 11:40pm

@elPiablo elPiablo self-assigned this Nov 19, 2023
]
},
{
"question": "How many methods are required by the ERC-20 standard to achieve the necessary functionality for a token contract?",
Copy link
Collaborator Author

@elPiablo elPiablo Nov 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@wolovim, @ happysingh pointed out to me the inaccuracy of this answer, and the text in the tutorial where it stems from. He wanted to do a PR and fix it, but is waiting for permissions for the repo.

Copy link
Contributor

@wolovim wolovim Nov 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@elPiablo this is a break from the current policy to give this level of repo permissions to Core Team members. not strictly opposed to adjusting that, but worth a quick convo in discord. if we don't grant further permissions, one alternative is to merge your work, then happy can follow up with a quick PR of his own, from his fork of the codebase.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @wolovim
I have six tasks on the list here, but we can settle for the first five. Item number 6 is a generic question about partners we need to answer at some stage ASAP, but this tutorial is not dependent on it to be merged (original PR message for clarity => #268 (comment))
You have quite a bit to get here, but I planned the commits before starting on them, so I hope you experience them cleanly and succinctly.
Have fun! 😄

Copy link
Collaborator Author

@elPiablo elPiablo Nov 21, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@elPiablo this is a break from the current policy to give this level of repo permissions to Core Team members. not strictly opposed to adjusting that, but worth a quick convo in discord. if we don't grant further permissions, one alternative is to merge your work, then happy can follow up with a quick PR of his own, from his fork of the codebase.

yeah, I just sent Happy a message and advised that course of action :) (https://discord.com/channels/883478451850473483/1171849215702483044/1176647550632210532)

@elPiablo
Copy link
Collaborator Author

elPiablo commented Nov 22, 2023

@wolovim just added an SEO friendly title and description that I had forgotten :7773888 . Should be good to go now.

Copy link
Contributor

@wolovim wolovim left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

upon skim, looks to be a solid iteration. approving without fine-toothed combing, so Happy can make his edits while its still on staging.

@elPiablo
Copy link
Collaborator Author

@Markkos89 can you debug the hydration errors on this branch, please? Cheers!

@Markkos89
Copy link
Collaborator

Markkos89 commented Nov 23, 2023

@Markkos89 can you debug the hydration errors on this branch, please? Cheers!

Note: with piablo saw that removing some comments that were invalidating some eslint rules in the QuizStatusChecker component the hydration error is not being triggered anymore

@elPiablo elPiablo merged commit aa13ec1 into staging Nov 23, 2023
2 checks 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