[Ellipsis] Prevent form submissions when is_open
is false
#91
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
Issue: #90
Implementation:
NewFormPage
component, add a new state variableisOpen
with its setter functionsetIsOpen
. Initialize it withtrue
. This will be used to control whether the form is open for submissions or not. The code should look like this:const [isOpen, setIsOpen] = useState(true);
onFormSubmit
function in theNewFormPage
component to check theisOpen
state before accepting submissions. IfisOpen
isfalse
, prevent form submission and display the specified message. The updated function should look like this:Report:
Add a state variable for is_open
I have successfully added a state variable foris_open
and updated the form submission function to check theis_open
state before accepting submissions. Ifis_open
isfalse
, the function prevents form submission and displays the specified message.Update form submission function
Updated theonFormSubmit
function in theNewFormPage
component to check theisOpen
state before accepting submissions. IfisOpen
isfalse
, the function prevents form submission and displays the specified message.Something look wrong?: If this Pull Request doesn't contain the expected changes, add more information to #90. Then, add the
ellipsis:create
label to try again. For more information, check the documentation.Generated with ❤️ by ellipsis.dev