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.

Sign up for GitHub

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

Jump to bottom

Revise tutorial "HOW TO WRITE & DEPLOY AN NFT" #13354

Open
wants to merge 13 commits into
base: dev
Choose a base branch
from
Open

Revise tutorial "HOW TO WRITE & DEPLOY AN NFT" #13354

wants to merge 13 commits into from

Conversation

aaronzshey
Copy link

I've made many edits to the tutorial "HOW TO WRITE & DEPLOY AN NFT". These include:

  • bump ethersjs to v6 and update methods accordingly
  • consolidate all alchemy packages into usage of alchemy-sdk and updated methods accordingly
  • convert all require statements to import methods
  • update outdated Solidity version (0.8.1 - 0.8.24) and update contract accordingly (namely, Countable.sol is deprecated)
  • update screen recordings of Alchemy dashboard

@github-actions github-actions bot added the content 🖋️ This involves copy additions or edits label Jul 8, 2024
@netlify Netlify
Copy link

netlify bot commented Jul 8, 2024 •
edited
Loading

❌ Deploy Preview for ethereumorg failed. Why did it fail? →

Name Link
🔨 Latest commit 6c6d030
🔍 Latest deploy log https://app.netlify.com/sites/ethereumorg/deploys/668e179a73f7080008855f19

@aaronzshey aaronzshey mentioned this pull request Jul 8, 2024
Open
2 tasks
@aaronzshey
Copy link
Author

closes #13353

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Aug 9, 2024

This issue is stale because it has been open 30 days with no activity.

@github-actions github-actions bot added the Status: Stale This issue is stale because it has been open 30 days with no activity. label Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wackerow wackerow Awaiting requested review from wackerow wackerow is a code owner

@corwintines corwintines Awaiting requested review from corwintines corwintines is a code owner

@pettinarip pettinarip Awaiting requested review from pettinarip pettinarip is a code owner

@minimalsm minimalsm Awaiting requested review from minimalsm minimalsm is a code owner

At least 0 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
content 🖋️ This involves copy additions or edits Status: Stale This issue is stale because it has been open 30 days with no activity.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@aaronzshey