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

My solution - Jason Courtois #17

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

Conversation

JasonCourtois
Copy link

@JasonCourtois JasonCourtois commented Dec 21, 2024

Please answer the following questions.

  1. What is your name?

Jason Courtois

  1. How long did you work on this project?

1 Hour and 50 minutes

  1. What is the most challenging part of the project?

For me, the most challenging part of this project was figuring out the best way to handle requests. I wanted to make sure that in this project no two tasks could share the same title. So figuring out the best way to handle each endpoint for my requests to prevent duplicate tasks was the most challenging part of the project.

  1. What did you learn from this project?

I learned how to make an API that can handle CRUD operations. I also have never worked directly with a SQLite database before in this way, only briefly through working with Django Python servers. I also learned the best error codes to use in different situations when a request fails such as when the user forgets to put in a title or if a task they are searching for doesn't exist.

  1. What would you like to add or improve in this project?

I think something that could be improved is the clarity of the endpoint names. I tried to make the endpoints as clear as possible, but I think I could improve both the organization of the endpoints as well as how each endpoint accepts data. I also think an interactive frontend with React would be a great addition to this project to make it user friendly.

  1. What is your feedback on this project?

Overall I think this was a fun project. I think it was very clear how the files were organized and I also feel like it gave enough information to guide me to the solution. By this I mean it already had the model file setup as well as the connection to the database, so we could focus on just getting the actual endpoints created.

  1. What is your suggestion for this project?

I think a suggestion I have for this project is actually another project idea. I have more frontend experience than backend, and I think an interesting project would be a frontend version of this idea. Instead of making endpoints, maybe there is a half complete frontend that we need to complete in order to display all the data from the sqlite server. This would give another option for those who have more frontend experience to show what they have learned.

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

Create a Todo App todo the following requirements:

  • It should be possible to create a task. - POST on /tasks (Needs title of task in body)
  • It should be possible to read a task. - GET on /tasks/read (Needs title in query)
  • It should be possible to mark a task as completed. - POST on /tasks/complete (Needs title of task in body)
  • It should be possible to update the title of task. - POST on /tasks/title (Needs oldTitle and newTitle in body)
  • It should be possible to delete a task. - DELETE on /tasks (Needs title in query)
  • It should be possible to list all tasks. - GET on /tasks
  • It should be possible to list all completed tasks. - GET on /tasks/complete
  • It should be possible to list all pending tasks. - GET on /tasks/pending

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

Ensure:

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

@JasonCourtois JasonCourtois changed the title feat: my solution My solution Dec 21, 2024
@JasonCourtois JasonCourtois changed the title My solution My solution - Jason Courtois Dec 21, 2024
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