Skip to content

docs(react-form): add type=submit in form-composition.md subscribe button #1521

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

Merged
merged 3 commits into from
May 23, 2025

Conversation

MrGeniusProgrammer
Copy link
Contributor

the docs does not clearly states that form.handleSubmit must be present in the subscribe button in form composition and will confuse many developers (and I did get confused). So with this it is clearly stated.

@MrGeniusProgrammer MrGeniusProgrammer changed the title Update form-composition.md with form.handleSubmit in the subscribe button docs(react-form): form-composition.md with form.handleSubmit in the subscribe button May 20, 2025
@MrGeniusProgrammer MrGeniusProgrammer changed the title docs(react-form): form-composition.md with form.handleSubmit in the subscribe button docs(react-form): added form.handleSubmit in form-composition.md subscribe button May 20, 2025
@MrGeniusProgrammer MrGeniusProgrammer changed the title docs(react-form): added form.handleSubmit in form-composition.md subscribe button docs(react-form): add form.handleSubmit in form-composition.md subscribe button May 20, 2025
Copy link
Member

@Balastrong Balastrong left a comment

Choose a reason for hiding this comment

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

Thank you!

@Balastrong Balastrong changed the title docs(react-form): add form.handleSubmit in form-composition.md subscribe button docs(react-form): add type=submit to in form-composition.md subscribe button May 23, 2025
@Balastrong Balastrong changed the title docs(react-form): add type=submit to in form-composition.md subscribe button docs(react-form): add type=submit in form-composition.md subscribe button May 23, 2025
@Balastrong Balastrong merged commit ed33c7b into TanStack:main May 23, 2025
1 check passed
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.

3 participants