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 Comments to server.js and main.js #42

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

schneedotdev
Copy link

Changes:

in package.json

  • added npm run start script

in main.js

  • added comments for functionality
  • added description of how the app works in java doc format
  • added authors to project

in server.js

  • added comments for functionality and each endpoint
  • fixed marked uncomplete endpoint by responding with "Marked UnComplete" instead of "Marked Complete"
  • set variable PORT to process.env.PORT || PORT and changed app.listen to app.listen(PORT, ()=>{ console.log(`Server running on port ${PORT}`) }) to avoid bug where environment variable is used but the console.log prints out the PORT value 2121

authors: Brian Schnee, Erika Teal, Bianca Togonon, Alondra Mora, Lucas Winkler, Alyssha Lewin, Jacob Asper, Dana Lee, Brenden D'Souza, Sebastian Ospina, Jesse Ranon, Alexis Aguilar, Pree Robertson

@schneedotdev
Copy link
Author

This PR should be assigned to Issue #41

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