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

Manabie Challenge: Mana Do task for Frontend Engineer applicants #261

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

Conversation

rysimbajon
Copy link

@rysimbajon rysimbajon commented May 12, 2022

Roberch Ryan Simbajon: Applicant: Frontend Engineer - Submitted (April 29, 2022)

Accomplished Tasks:

Update of todo status
Deleting of todo status
Fixed a bug when adding task, it doubled upon enter
LocalStorage implementation

How to run tests:

npm i (install first dependencies used)
yarn start:frontend

Things to add if given more time:

Improve UI design, enhancements of layout, appearance, user ergonomics in interactive such web app
Improve more in TypeScript implementions, coding standards(it's my first time to use TypeScript, learn and exam in the same time, I want to learn more in this platform, so while learning TypeScript, I just applied my learning concepts from React.js, which is very challenging in my part.)

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