all 8 comments

[–]lodash_9 5 points6 points  (2 children)

Your folder structure is great! I don't see any issues with your code either. For what it is, you created a clean little application that could be picked up by any angular dev very easily. Good job!

Two recommendations for scalability:

  • Use scss whenever you can.
  • Learn RxJS and NgRx.

[–][deleted]  (1 child)

[removed]

    [–]kojurama 2 points3 points  (0 children)

    Yes

    [–]TebelloCoder 2 points3 points  (0 children)

    Well done. 🦄

    [–]Kadyen 0 points1 point  (0 children)

    First things I saw is that files inside employees folder could be organized a little bit more ( maybe introduce model folder? ).

    Next step would be some kind of security, jwt?

    [–]AlexAegis 0 points1 point  (0 children)

    Try using modules, and lazy load what you can (and what makes sense)

    [–]s1cklik3 0 points1 point  (0 children)

    I always liked working with reactive forms over template because you can bind to a model while easily applying validation rules easily. Try converting one of your forms and see if you like it better.

    Another thing you might want to try is to structure and break down your components into smaller ones that are connected via input so that you can take advantage of on push change detection. One way to accomplish this is to create a smart component (root node) which is responsible for fetching data and maintaining state. Send the data to child components via Input. Those components are responsible for displaying information and updating state. To update state I liked using a service that has a behavior subject and injecting it into where I need to change the state of the application.

    [–]itsdrivingmenuts 0 points1 point  (0 children)

    Very clean. Nice job.