Skip to content
This repository was archived by the owner on Mar 17, 2026. It is now read-only.

front end test said - #7

Open
saidkholov wants to merge 1 commit into
ClearPointNZ:mainfrom
saidkholov:frontend_test_said_kholov
Open

front end test said#7
saidkholov wants to merge 1 commit into
ClearPointNZ:mainfrom
saidkholov:frontend_test_said_kholov

Conversation

@saidkholov

@saidkholov saidkholov commented May 8, 2024

Copy link
Copy Markdown

I spent about 1.5 hours on the task. There are definitely a lot more things that could be improved:

  • Potentially add more unit tests, I thought a couple of unit tests should be enough to demonstrate my ability
  • Add linting(Eslint+prettier)
  • Convert the codebase to TS
  • Breakdown to components
  • Storybook

and etc.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant