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

zksync 101 content #14

Closed
wants to merge 61 commits into from
Closed

zksync 101 content #14

wants to merge 61 commits into from

Conversation

uF4No
Copy link
Contributor

@uF4No uF4No commented Apr 12, 2024 •
edited
Loading

What 💻

  • Adds first day content
  • Creates partials for tutorials
  • Adds networkAdder component

Why ✋

  • new content

Evidence 📷

Include screenshots, screen recordings, or console output here demonstrating that your changes work as intended

⚠️ DISCLAIMER:

  • 101 quickstart and ERC020 tutorials are available in both Atlas and Remix
  • Paymaster scripts on Remix are blocked because Remix does not support zksync-ethers yet. The current content is just a placeholder that I'd update once unblocked.

dutterbutter and others added 21 commits April 1, 2024 21:09
@dutterbutter
feat: initial quickcode guide for deploying contract using hardhat or…
c7bc343 
… foundry
@dutterbutter
feat: initial quickcode guide for deploying contract using hardhat or…
fd75318 
… foundry
@dutterbutter
chore: update navigation
cf516c3
@dutterbutter
fix: update nav link
f2b794a
@itsacoyote
feat: create components for markdown content
eaa957e
@dutterbutter
chore: Foundry content, and minor improvements
c238360
@dutterbutter
chore: Foundry content, and minor improvements
308e6ae
@dutterbutter
chore: rename dir to quick-start instead of quick-code
f471161
@dutterbutter
feat: adds deploy factories and testing, still some issues to resolve…
03aa7d8 
… with components
@dutterbutter
feat: update upgrading directories and content
2bda316
@dutterbutter
debug: adds dev tools for improved debugging
583f2d7
@dutterbutter
fix: fix content switch basepath issue
4ebae9f
@dutterbutter
chore: resolve conflicts
d99e4c6
@dutterbutter
chore: fix spelling
03dda78
@dutterbutter
chore: fix nav in contributing
ccf0cfe
@dutterbutter
chore: content updates
9cf02d5
@dutterbutter
chore: content updates
6fb91aa
@dutterbutter
chore: content updates for upgrades
f00ce98
@dutterbutter
Merge branch 'staging' of github.com:matter-labs/zksync-docs into db/…
8f916d6 
…quickcode-guide
@dutterbutter
chore: update with staging
e7da9fd
@uF4No
feat: 101 content
a8ba6ea
@dutterbutter
Copy link
Contributor

@uF4No can you resolve conflicts so we can make use of the preview link please.

@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 15, 2024 •
edited
Loading

Visit the preview URL for this PR (updated for commit c228908):

https://zksync-docs-staging-5eb09--pr14-feat-zksync-101-jp1p982h.web.app

(expires Thu, 02 May 2024 16:17:08 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: bfaafba5fa82d4f63473aaa76a21fabf1fbb3a11

itsacoyote
itsacoyote requested changes Apr 23, 2024
View reviewed changes
components/content/NetworkAdder.vue Outdated Show resolved Hide resolved
components/content/NetworkAdder.vue Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/1.zksync-overview.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/2.connect-zksync.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/3.faucet.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/_quickstart/_remix_deploy_contract.md Outdated Show resolved Hide resolved
content/_partials/enable-remix-zksync-plugin.md Outdated Show resolved Hide resolved
content/_partials/environment-setup-with-zksync-cli.md Outdated Show resolved Hide resolved
nuxt.config.ts Outdated Show resolved Hide resolved
public/images/metamask_logo.svg Outdated Show resolved Hide resolved
dutterbutter
dutterbutter previously requested changes Apr 24, 2024
View reviewed changes
Copy link
Contributor

@dutterbutter dutterbutter left a comment

Choose a reason for hiding this comment

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

Love the buttons for quick usage!

content/00.build/00.zksync-101/1.zksync-overview.md Show resolved Hide resolved
content/00.build/00.zksync-101/4.quickstart.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/5.erc20-token.md Show resolved Hide resolved
content/00.build/00.zksync-101/6.paymasters-introduction.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/_erc20_tutorial/_atlas_erc20_tutorial.md Show resolved Hide resolved
content/00.build/00.zksync-101/_erc20_tutorial/_remix_erc20_tutorial.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/_erc20_tutorial/_remix_erc20_tutorial.md Outdated Show resolved Hide resolved
content/00.build/00.zksync-101/_paymaster_intro/_remix_paymaster_intro.md Outdated Show resolved Hide resolved
content/_partials/compile-solidity-contracts.md Outdated Show resolved Hide resolved
content/_partials/environment-setup-with-foundry.md Outdated Show resolved Hide resolved
itsacoyote
itsacoyote requested changes Apr 25, 2024
View reviewed changes
Copy link
Contributor

@itsacoyote itsacoyote left a comment

Choose a reason for hiding this comment

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

Appreciate putting up with all of my comments, thank you so much for all this work! ❤️
There's just one last line edit to change and everything else looks great 👍🏻

nuxt.config.ts Outdated Show resolved Hide resolved
@itsacoyote itsacoyote dismissed dutterbutter’s stale review April 26, 2024 14:48

OoO and changes have been completed.

@uF4No
Copy link
Contributor Author

uF4No commented Apr 29, 2024

moving changes to #34

@uF4No uF4No closed this Apr 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dutterbutter dutterbutter dutterbutter approved these changes

@itsacoyote itsacoyote itsacoyote approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@uF4No @dutterbutter @itsacoyote