From 45fef12590342704742657d15352594ee199785c Mon Sep 17 00:00:00 2001 From: Stedoss <29103029+Stedoss@users.noreply.github.com> Date: Tue, 1 Nov 2022 20:21:15 +0000 Subject: [PATCH] Update readme todos --- README.md | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/README.md b/README.md index 418cb6b..a1dceb9 100644 --- a/README.md +++ b/README.md @@ -111,6 +111,10 @@ There are three clear entities that could be split out into microservices (they Only unit tests are included, which are good however integration and automated tests would be ideal for production software. +### Pagination + +Pagination is something that I've not really had a chance to look into (apart from a bit of it in Laravel), so it would have been nice to deal with it here to learn a bit more about it. And make the index page perform much nicer with the searches. + ### DTOs Returning raw database objects as responses isn't really ideal (and I found an issue with many-to-many relationships which causes an infinite loop in the JSON parser!), so it would be ideal to use DTOs when returning objects from the API. @@ -131,8 +135,4 @@ More for the frontend, but backend doesn't have the best either. More checks mak ### Add unit tests for FE -Even if it's just a few, it will show off how I unit test for the frontend. - -### Cors? - -Cors may be an issue here. \ No newline at end of file +Even if it's just a few, it will show off how I unit test for the frontend. \ No newline at end of file