Skip to content

Use updateErrors so fields are touched and server errors visible#1

Open
tabby-or-not wants to merge 2 commits intofurious-luke:masterfrom
tabby-or-not:bug/display-server-errors
Open

Use updateErrors so fields are touched and server errors visible#1
tabby-or-not wants to merge 2 commits intofurious-luke:masterfrom
tabby-or-not:bug/display-server-errors

Conversation

@tabby-or-not
Copy link

Errors thrown during submit currently aren't visible as the fields are not "touched"
If we use updateErrors instead of setErrors, these fields will be "touched" and the errors accessible through the proxy.

@codecov-commenter
Copy link

Codecov Report

Merging #1 (7c91490) into master (b55562b) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #1   +/-   ##
=======================================
  Coverage   95.87%   95.87%           
=======================================
  Files           7        7           
  Lines         194      194           
  Branches       34       34           
=======================================
  Hits          186      186           
  Misses          8        8           
Impacted Files Coverage Δ
src/useForm.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b55562b...7c91490. Read the comment docs.

@tabby-or-not tabby-or-not marked this pull request as draft May 27, 2021 04:56
@tabby-or-not tabby-or-not marked this pull request as ready for review May 28, 2021 00:55
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.

2 participants