Skip to content

fix: Do not call e.preventDefault()#3

Merged
Almouro merged 1 commit intomasterfrom
fix/AvoidPreventDefault
Mar 27, 2018
Merged

fix: Do not call e.preventDefault()#3
Almouro merged 1 commit intomasterfrom
fix/AvoidPreventDefault

Conversation

@chdeps
Copy link
Copy Markdown

@chdeps chdeps commented Mar 26, 2018

Calling submitForm instead of handleSubmit to bypass the call to e.preventDefault()

@chdeps chdeps force-pushed the fix/AvoidPreventDefault branch from b30ce4f to 28d0353 Compare March 26, 2018 18:10
@chdeps chdeps force-pushed the fix/AvoidPreventDefault branch from 28d0353 to 83863f8 Compare March 26, 2018 18:12
@chdeps chdeps changed the title Do not call e.preventDefault() fix: Do not call e.preventDefault() Mar 26, 2018
@coveralls
Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 18

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 95.556%

Totals Coverage Status
Change from base Build 11: 0.0%
Covered Lines: 65
Relevant Lines: 67

💛 - Coveralls

@Almouro Almouro merged commit ba4563a into master Mar 27, 2018
@Almouro Almouro deleted the fix/AvoidPreventDefault branch March 27, 2018 06:55
@Almouro
Copy link
Copy Markdown
Contributor

Almouro commented Mar 27, 2018

Thanks @chdeps for the PR!

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