Skip to content
This repository has been archived by the owner on Jan 26, 2024. It is now read-only.

Add code to prevent double submission #34

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gewenyu99
Copy link

The form is not disabled as the first submission is handled. This allows the form to be submitted twice if you have fast fingers like me :)

This PR attempts to address this problem using a hook and a fieldset disable. Not sure if this is good practice or antipattern.

@vercel
Copy link

vercel bot commented Mar 14, 2022

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/appwrite/todo-with-react/5n2h1C4yAwi9oVq6LW2jAZT9YjPB
✅ Preview: https://todo-with-react-git-fork-gewenyu99-fix-double-s-1b17a0-appwrite.vercel.app

@brandonroberts
Copy link
Contributor

I'd recommend just disabling the input field until its finished processing

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants