Skip to content
This repository was archived by the owner on Mar 11, 2024. It is now read-only.

Remove event.preventDefault call #39

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

Remove event.preventDefault call #39

wants to merge 1 commit into from

Conversation

azappella
Copy link

Problem:

The onProfileFormSubmit function makes a redundant call to event.preventDefault()

  • event.preventDefault is already called inside handleFormSubmit (src/user/ui/ProfileForm.js)
  • event is never passed from the function which causes an error when trying to update the user name

Solution: removed the redundant event.preventDefault call
@azappella
Remove event.preventDefault call
022f366 
Problem: The onProfileFormSubmit function makes a redundant call to event.preventDefault() - event.preventDefault is already called inside handleFormSubmit (src/user/ui/ProfileForm.js) - event is never passed from the function which causes an error when trying to update the user name Solution: removed the redundant event.preventDefault call
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

1 participant

@azappella