Fix: return onSubmit promise result in submission

[mmartinsky]

Fixes #2419 , maintaining the value of the user's promise.

[vercel]

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

🔍 Inspect: https://zeit.co/jared/formik-docs/a2o45q8zz
✅ Preview: https://formik-docs-git-fork-mmartinsky-master.jared.now.sh

[codesandbox-ci]

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 366dd75:

Sandbox	Source
Formik TypeScript Playground	Configuration
magical-mendel-gjocs	Issue #2419

[johnrom]

Looks good to me barring some minor changes re: dev environment.

I'm not super excited with the formatting changing for the whole file, so maybe it'd be best to only commit those changes which are relevant to this PR. But maybe @jaredpalmer has some thoughts on that.

[jaredpalmer]

Can we only format the changed lines?

[johnrom]

If the formatting comes from commit hooks that might be a problem. In sourcetree there is the option to "Bypass commit hooks". I don't think this exists in Github Desktop. If it comes from prettier / vscode then it is as simple as "ctrl/cmd+shift+p -> Save without formatting" or disabling formatting temporarily.

[mmartinsky]

Thanks for the quick responses - pushed up a new version with just the needed lines modified, formatting was from User level settings on my VSCode - should be taken care of now though

[mmartinsky]

Any word on when this might get merged? Would love to switch back off of my forked package

[cubeghost]

Also interested in seeing this get merged!