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

Feat: Create MovieFan - search the movie and check information about it #666

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

Conversation

1devill
Copy link

@1devill 1devill commented Dec 13, 2024

About my process of thinking when developing it

Besides required React and TS:

  • Thought to not use any 3rd party state management for such a small project. It was enough for me to work with useState() and useContext() and to have a more lightweight bundle after all.
  • Wanted to try out Vite bundler due to it's popularity, modern approach and cool features like per route bundling and prebundling deps.
  • Decided to style everything with styled-components as your company's main styling tool.
  • Configured eslint/tslint for the project, used prettier also.
  • For unit testing I went with jest and react-testing-library. Covered a couple of tests to show my approach to that.

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