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

Added async/await to fix firefox issue + other things #16

Merged
merged 7 commits into from
Jan 18, 2020

Conversation

zapdos26
Copy link
Contributor

@zapdos26 zapdos26 commented Jan 15, 2020

Fixes Firefox issue of not adding calls/notes.
Prevent notes from being empty;
Removed window reload upon adding note/call;
Added form reset on adding a note/call;
Added refresh of notes on adding a call;
Added await to getNotes in deleteCall();
Added </div> that was missing in the admin.html

Currently the reset of the addCall causes the validator to run (this occurs even with reloading the page). This will need to be looked into for Firefox.

lramos15
lramos15 previously approved these changes Jan 15, 2020
Copy link
Contributor

@lramos15 lramos15 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@ddbruce ddbruce left a comment

Choose a reason for hiding this comment

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

Please see the comments I left thus far. I'll take another look when those changes are made. Thanks!

public/js/admin.js Outdated Show resolved Hide resolved
Renamed postServer to postToServer;
Prevent notes from being empty;
Removed window reload upon adding note/call;
Added form reset on adding a note/call;
Added refresh of notes on adding a call;
Added await to getNotes in deleteCall();
Added </div> that was missing in the admin.html

Currently the reset of the addCall causes the validator to run (this occurs even with reloading the page). This will need to be looked into for FireFox.
@zapdos26 zapdos26 changed the title Added async/await to fix firefox issue Added async/await to fix firefox issue + other things Jan 16, 2020
package.json Outdated Show resolved Hide resolved
@codeclimate
Copy link

codeclimate bot commented Jan 17, 2020

Code Climate has analyzed commit 7f6cf1e and detected 0 issues on this pull request.

View more on Code Climate.

Copy link
Member

@ddbruce ddbruce left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks, @zapdos26!

@lramos15 lramos15 merged commit 210f310 into master Jan 18, 2020
@ddbruce
Copy link
Member

ddbruce commented Jan 23, 2020

Fixes #15

@ddbruce ddbruce deleted the add-async-await branch February 1, 2020 17:13
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