Skip to content

Working example for building forms with Next.js #32538

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

Closed
wants to merge 3 commits into from

Conversation

MaedahBatool
Copy link
Contributor

Feature

Documentation / Examples

  • Make sure the linting passes by running yarn lint

@leerob
Copy link
Contributor

leerob commented Dec 16, 2021

It looks like you committed a submodule - could you add the files directly here? 🙏

@MaedahBatool
Copy link
Contributor Author

It looks like you committed a submodule - could you add the files directly here? 🙏

@leerob Actually the original folder was on my GitHub profile and I cloned it directly in the examples folder. I think that's the issue. Let me fix this. 🙈

@MaedahBatool
Copy link
Contributor Author

MaedahBatool commented Dec 16, 2021

next-forms.zip

Here are attached files in zipped form.

@leerob
Copy link
Contributor

leerob commented Dec 16, 2021

Actually the original folder was on my GitHub profile and I cloned it directly in the examples folder. I think that's the issue.

Could you update this PR to have the files committed directly, instead of the submodule where you cloned to?

Then, you should be able to see the files changed in the "Files changed" tab after this 😄

@MaedahBatool
Copy link
Contributor Author

Actually the original folder was on my GitHub profile and I cloned it directly in the examples folder. I think that's the issue.

Could you update this PR to have the files committed directly, instead of the submodule where you cloned to?

Then, you should be able to see the files changed in the "Files changed" tab after this 😄

Ok makes sense. Thanks for the help. I just need to revise the code files a bit and once that's done I'll commit the final version here. :)

@MaedahBatool
Copy link
Contributor Author

I've submitted a new PR for that: #32669

@vercel vercel locked as resolved and limited conversation to collaborators Jan 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
examples Issue was opened via the examples template.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants