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

Assignment 3 #3

Open
JoeBlanc opened this issue Sep 29, 2023 · 1 comment
Open

Assignment 3 #3

JoeBlanc opened this issue Sep 29, 2023 · 1 comment

Comments

@JoeBlanc
Copy link
Owner

link demo: https://assignment3-cyan.vercel.app/
link repo: https://github.com/JoeBlanc/df-frontend-2023/tree/main/assignment-3

@ngolapnguyen
Copy link

ngolapnguyen commented Oct 8, 2023

Requirements

  • All core functionalities present
  • Proper TypeScript implementation
  • No eslint warnings & errors
  • Husky & lint-staged integration

Final result: ❌. Typescript is barely integrated.

Feedbacks

  1. Issues with pagination:
    • No visual cue for current page
    • Pages should be re-calculated on search query change. Right now they seem to be fixed
image
  1. Typescript is barely integrated. Across the app you are still using Javascript & implicit any type. You can refer to this repo: https://github.com/longphanquangminh/df-frontend-2023/tree/main/assignment-3. It's a good reference on how we should be approaching this assignment.

You can update this line to true to see the places you are using implicit any:

"noImplicitAny": false,

Try defining a type/interface to fix all the issues that pop up after you set the config to true. You'll get a better understanding of how Typescript works 👍

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

No branches or pull requests

2 participants