Open faroukellouze opened 3 years ago
Done
[ ] You are still using the entity in your controller, not the DTO https://github.com/BlEmnaa/clinisys-app/blob/6e638b116fde6c94a4c9f85c2ce6e05fd941a225/src/main/java/com/clinisys/controller/ClinsysController.java#L76
[ ] The controller name is still non-informative. What does it mean ClinsysController? You should pick a more significant name
[ ] The URIs used in your mappings do not follow good practices. For example this one products/getAll https://github.com/BlEmnaa/clinisys-app/blob/6e638b116fde6c94a4c9f85c2ce6e05fd941a225/src/main/java/com/clinisys/controller/ClinsysController.java#L45
[ ] This method should return a Product, not List<LinkedHashMap<String,String>> https://github.com/BlEmnaa/clinisys-app/blob/6e638b116fde6c94a4c9f85c2ce6e05fd941a225/src/main/java/com/clinisys/controller/ClinsysController.java#L70
Please follow the videos we've given you to solve all this issue. The code you've written doesn't respect many of the concepts discussed in the tutorial
Consider using DTOS in your controller(ProduitDTO instead of Produit). Also, change the name of your controller class.