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

Submission for intern assesment #15

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Jaidoescode
Copy link

Please answer the following questions.

  1. What is your name?
    Jaideep Singh

  2. How long did you work on this project?
    I worked on this project for about 90 minutes

  3. What is the most challenging part of the project?
    This project has been a great experience and trying to troubleshoot Sequelize sync issues has been a hurdle. Learning and fixing the database connections and also handling the edge cases was a bit challenging.

  4. What did you learn from this project?
    I learned a lot about working with Node.js and Express, particularly in building RESTful APIs. I also gained hands-on experience with Sequelize ORM for interacting with the SQLite database. Additionally, I became more comfortable with error handling and ensuring data integrity across various database operations.

  5. What would you like to add or improve in this project?
    A more realistic model would not allow public access. If I were to suggest one thing, I would consider user authentication and making the frontend of the project more welcoming and user friendly.

  6. What is your feedback on this project?
    I think the project was well-designed for learning basic CRUD operations using Express and Sequelize. The API functionality is solid, but I feel the project could benefit from additional features like user authentication.

  7. What is your suggestion for this project?
    N/A

  8. Could you mark which of the following requirements you have completed?

Create a Todo App todo the following requirements:

  • [Done] It should be possible to create a task. // using the POST route makes tasks
  • [Done] It should be possible to read a task. // GET route
  • [Done] It should be possible to mark a task as completed. // PATCH route
  • [Done] It should be possible to update the title of task. // PUT route
  • [Done] It should be possible to delete a task. // DELETE route
  • [Done] It should be possible to list all tasks. //GET route to pull all tasks from the database
  • [Done] It should be possible to list all completed tasks. // GET route with query parameter
  • [Done] It should be possible to list all pending tasks.

Once competed, you can submit a pull request to the original repository.

Ensure:

  • [Done] The code is clean and readable.
  • [Done] The code is well-structured.
  • [Done] The code is well-commented.
  • [Done] Only the required files are committed.
  • [Done] Answer the questions in the pull request template.

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