-
Notifications
You must be signed in to change notification settings - Fork 0
Open
Description
Hi @franjurinec!
I reviewed your Full stack course project. You do not have to do anything else to receive the credits. If you like, you can use the comments in this or future projects.
Application
I tested only the view for all users as I did not have the password for the admin view. The functionality seemed to work without any problems. Nice that you have studied content outside the course content, e.g., the payment functionalities.
Code and repository
- README.md clear and informative
- Cypress is used for testing, nice!
- EditProductFrom and AddProductForm are basically the same, could one form serve both use cases?
- Naming functions and variables is reasonable and code is well structured
Some ideas to develop the app further, if you would like to receive 10 credits for the course:
- Each item belong at least one category and these category groups could be shown on the main page. By clicking one group the user could see the items from this category.
- Items can be sorted by prize or/and by time as the item was downloaded
- Items can be searched by, for example, categories and name.
Overall, well done!
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels