Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixing form submit, resolves #14 #22

Merged
merged 1 commit into from Feb 19, 2019
Merged

Fixing form submit, resolves #14 #22

merged 1 commit into from Feb 19, 2019

Conversation

hutchgrant
Copy link
Member

Prevent form from reloading the page on enter.

Copy link
Member

@thescientist13 thescientist13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hey @hutchgrant .

Is it possible to undo the formatting only related changes? For reviewing a PR it makes it a bit harder to just see the specific changes for the fix / change itself.

Ideally some sort of tool would keep this all in line, but for this project, as an example, doesn't really have that yet. But if something like prettier were to be added, I think it would be best to add it and format all the files in its own PR.

@thescientist13 thescientist13 added the bug Something isn't working label Nov 6, 2018
hutchgrant added a commit that referenced this pull request Nov 8, 2018
thescientist13 pushed a commit that referenced this pull request Nov 10, 2018
* Fixing form submit, resolves #14

* Adding karma unit tests, resolves #12

* Removing fix for #22 and updating karma conf

* Removing reports before each test
Copy link
Member

@thescientist13 thescientist13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hutchgrant
In addition to undoing formatting, you will want to rebase your PR here against the latest changes just merged into master.

@thescientist13
Copy link
Member

@hutchgrant
Looks like there is a merge conflict here. Other than that, this LGTM.

Copy link
Member

@thescientist13 thescientist13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks @hutchgrant 👍

@thescientist13 thescientist13 merged commit 9b0f035 into master Feb 19, 2019
@thescientist13 thescientist13 deleted the form-submit-fix branch February 19, 2019 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants