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

Thoughts on the Implementation of the Project #17

Open
jmatembu opened this issue Jan 30, 2020 · 0 comments
Open

Thoughts on the Implementation of the Project #17

jmatembu opened this issue Jan 30, 2020 · 0 comments

Comments

@jmatembu
Copy link
Collaborator

jmatembu commented Jan 30, 2020

@MuhweziDeo

Thank you for the time you have put in this project. I would like to share my thoughts and general assessment of your input on the project.

Overall Remark:

Your code is good, and well organized. It generally follows good practices. With these skills you can practically contribute to projects of different levels of complexity.

What I liked:

  • Practical git workflow for teams
  • File and Folder organization
  • Use of Repository pattern showed some practical knowledge of Design Patterns
  • Good knowledge Events
  • Proper of use of Middleware
  • Simple use of user roles using Model methods is_admin() and isStaff()
  • General extraction of validation logic to Form Request objects.

What I kinda did not like:

  • The table structure for recording sale of items and service. Introduces multiple complexities.
  • Minor cases of possible over-engineering for a simple project. A good example here is using of Form Request object for validation before creating service is not necessary since you are validating just two fields.
  • No single test written
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

No branches or pull requests

1 participant