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

docs(fwdx.xyz): Add fwdx quickstart tutorial #181

Merged
merged 8 commits into from
Jun 26, 2023

Conversation

SirWeb3Wizard
Copy link
Contributor

@SirWeb3Wizard SirWeb3Wizard commented Jun 24, 2023

Pull request checklist

Use the following template to make sure your PR fits the ConsenSys documentation standard.

Before creating the PR

Make sure that:

Describe the change

Updated the links that went to blockscout and changed them to the goerli.lineascan.build

Issue fixed

No current issue exists

Impacted parts

For content changes:

  • [ x] Documentation content
  • Documentation page organization

For tool changes:

  • Build and QA tools
  • Node dependencies and JavaScript
  • GitHub integration

After creating your PR and tests have finished

Make sure that:

  • You've fixed any issues raised by the tests.
  • You've previewed your changes on Vercel below.

@vercel
Copy link

vercel bot commented Jun 24, 2023

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

Name Status Preview Comments Updated (UTC)
doc-zk-evm ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jun 25, 2023 2:56am

@emilyJLin95
Copy link
Contributor

@SirWeb3Wizard you've got a few errors

  1. Can you close all your <img tags with />
  2. Can you add FWDX to project-words.txt
  3. You misspelled Click as Cick

And then you can check that these work by running npm run build and npm run lint

@SirWeb3Wizard
Copy link
Contributor Author

@SirWeb3Wizard you've got a few errors

  1. Can you close all your <img tags with />
  2. Can you add FWDX to project-words.txt
  3. You misspelled Click as Cick

And then you can check that these work by running npm run build and npm run lint

Thanks @emilyJLin95. Have fixed it.

Copy link
Contributor

@emilyJLin95 emilyJLin95 left a comment

Choose a reason for hiding this comment

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

LGTM

@emilyJLin95 emilyJLin95 merged commit ec94311 into Consensys:main Jun 26, 2023
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