-
Notifications
You must be signed in to change notification settings - Fork 8
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
Practice app/frontend/categories page 204 #205
Practice app/frontend/categories page 204 #205
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After running the code in my local, I have observed that when entering an invalid category in Add Category field, the category is added as undefined. I think it should show an error and do not add anything to the categories list.
Besides that I think everything looks amazing, good work!
I solved the problem and added suggesting feature property to the categories. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed the whole work you did. I fetched and pulled the branch on my local repo. Then, I tested the functionalities. I encountered with the error about two way suggestions (suggesting sport when written sports and vice versa). @mbatuhancelik solved this problem with the latest commit. Currently, I couldn't see anyother problem and the functionality works as expected. I am approving the PR and merging it to the master branch.
As mentioned in #204, frontend page for categories is implemented and ready to be used.