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

Working example for building forms with Next.js #32669

Merged
merged 14 commits into from Jan 17, 2022

Conversation

MaedahBatool
Copy link
Contributor

This PR adds a working example of Next.js forms that I wrote with the Next.js forms guide.

Feature

Documentation / Examples

  • Make sure the linting passes by running yarn lint

Fixes #32538

@ijjk ijjk added examples Issue/PR related to examples created-by: Next.js docs team PRs by the Next.js docs team labels Dec 20, 2021
@MaedahBatool
Copy link
Contributor Author

Hi @leerob, 👋
I created this new PR. Please review it. Thank you!

examples/next-forms/package.json Outdated Show resolved Hide resolved
examples/next-forms/package.json Outdated Show resolved Hide resolved
examples/next-forms/package.json Outdated Show resolved Hide resolved
examples/next-forms/pages/index.js Outdated Show resolved Hide resolved
examples/next-forms/pages/js-form.js Outdated Show resolved Hide resolved
examples/next-forms/pages/js-form.js Outdated Show resolved Hide resolved
MaedahBatool and others added 6 commits December 21, 2021 15:58
Co-authored-by: Lee Robinson <me@leerob.io>
Co-authored-by: Lee Robinson <me@leerob.io>
Co-authored-by: Lee Robinson <me@leerob.io>
Co-authored-by: Lee Robinson <me@leerob.io>
Co-authored-by: Lee Robinson <me@leerob.io>
Co-authored-by: Lee Robinson <me@leerob.io>
@MaedahBatool MaedahBatool self-assigned this Jan 6, 2022
@MaedahBatool
Copy link
Contributor Author

@leerob, can we please get this merged if all looks fine? TIA :)

examples/next-forms/README.md Outdated Show resolved Hide resolved
examples/next-forms/README.md Show resolved Hide resolved
examples/next-forms/pages/no-js-form.js Outdated Show resolved Hide resolved
@MaedahBatool
Copy link
Contributor Author

@leerob please can you now take a final look. I hope it looks sorted now. 🤞

leerob
leerob previously approved these changes Jan 14, 2022
@balazsorban44 balazsorban44 merged commit 790db2c into vercel:canary Jan 17, 2022
natew pushed a commit to natew/next.js that referenced this pull request Feb 16, 2022
* feat: Forms example

* docs: deploy button

* Update examples/next-forms/package.json

Co-authored-by: Lee Robinson <me@leerob.io>

* Update examples/next-forms/package.json

Co-authored-by: Lee Robinson <me@leerob.io>

* Update examples/next-forms/package.json

Co-authored-by: Lee Robinson <me@leerob.io>

* Update examples/next-forms/pages/js-form.js

Co-authored-by: Lee Robinson <me@leerob.io>

* Update examples/next-forms/pages/js-form.js

Co-authored-by: Lee Robinson <me@leerob.io>

* Update examples/next-forms/pages/index.js

Co-authored-by: Lee Robinson <me@leerob.io>

* Update examples/next-forms/README.md

Co-authored-by: Lee Robinson <me@leerob.io>

* Update comments formatting

* Improve docs for correct examples format.

* Lint tests

Co-authored-by: Lee Robinson <me@leerob.io>
@vercel vercel locked as resolved and limited conversation to collaborators Feb 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Next.js docs team PRs by the Next.js docs team examples Issue/PR related to examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants