Skip to content

Update dependencies and fix errors #103

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Update dependencies and fix errors #103

wants to merge 3 commits into from

Conversation

skoblenick
Copy link

The related tutorial is nicely done however once it becomes dated it can be a hurdle for newcomers to debug a technology stack they have little experience with. Most developers need to know the latest API's and best practices because it is where they're generally starting when looking at with a boilerplate repo. Debugging many of GraphQL errors aren't fun when you have to know the order of execution, how the stack generally works and how to track down the sometimes cryptic error backs.

This PR will close out several outstanding issues and PR's made by others. In addition the PR adds:

  • a NPM test script to run the server and test execution script in parallel
  • an ignore-file to nodemon when test files are updated as test changes do not impact the code execution or compilation

- version bump all dependencies to latest (with fixed versions)
- fix GraphQL error when looking up users for messages
- fix deprecated usages of findById; updated to findByPk
- add Docker compose file to launch PostgreSQL and Adminer

Closes #48, Closes #60, Closes #81, Closes #85, Closes #87, Closes #89,
Closes #97, Closes #99, Closes #101
@skoblenick
Copy link
Author

I am making assumption the env vars: SECRET, DATABASE_USER, DATABASE_PASSWORD and PORT are defined within the Travis project settings, otherwise they will need to be added to the NPM script.

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.

1 participant