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

Update Project #45

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

Update Project #45

wants to merge 7 commits into from

Conversation

alexvargashn
Copy link

@alexvargashn alexvargashn commented Jan 4, 2024

Update the complete project

The project was update to the latest express modern way to handle rest api.
The following dependencies was added:

  • cors

The project structure was update so:

  • controllers
  • data
  • models
  • routes
    app.js

Controllers:
The functions that call the logic for every request.

Data
The file that contains the data about quotes.

Models
The classes, in this case the Server class to handle the server global logic.

Routes.
The files that handles the functions, middlewares, etc for every route called.

app.js
The main entry for the application.

  • Note: Aditionaly
    I needed something like /v1/quotes/by/{id} feature from this api, so I took a time for add it. It works giving to you a specific quote by idIt works giving to you and specific quote by id

Copy link

vercel bot commented Jan 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
breaking-bad-quotes ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 4, 2024 5:22pm

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