From bb2d3a5104029fdc0460f4d484f30f8d6b703fe6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Str=C3=B8m-Andersen?= Date: Fri, 18 Sep 2026 21:06:50 +0200 Subject: [PATCH 1/2] docs: add project choices and reasoning --- README.md | 12 ++++++++++++ 1 file changed, 12 insertions(+) diff --git a/README.md b/README.md index e4a5f7e..b788889 100644 --- a/README.md +++ b/README.md @@ -55,6 +55,18 @@ Before a pull request can be merged: after being approved, ask for another review. This is deliberate: what was approved is what gets merged. +## Project choices and reasoning + +A choice we have made is to drop creating a favorites page where only books that you have +saved as a favorite would be shown in a grid similar to the one on the "front page" ("search page"). +We made this choice because we found that the API did not support searching for multiple +books at once, which would have required an unnecessarily large number of API calls to +achieve this without saving data - something the project description specified that we should not do. + +Another choice we have made is to include some external icons. All of these +are found on [https://lucide.dev/icons](https://lucide.dev/icons), which is a well-known open-source icon library. +We think this is a nice addition to the plain CSS the app is otherwise built with. + ## Use of AI in this project We have used Claude (through Claude Code) as a tool during development. In From 6b5de018eac2264c45d3d3c14f35329e1373d137 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Str=C3=B8m-Andersen?= Date: Fri, 18 Sep 2026 21:50:19 +0200 Subject: [PATCH 2/2] docs: fix API limitation reasoning Co-authored-by: Martin Alexander Kaminski --- README.md | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/README.md b/README.md index b788889..b84ca41 100644 --- a/README.md +++ b/README.md @@ -59,8 +59,8 @@ Before a pull request can be merged: A choice we have made is to drop creating a favorites page where only books that you have saved as a favorite would be shown in a grid similar to the one on the "front page" ("search page"). -We made this choice because we found that the API did not support searching for multiple -books at once, which would have required an unnecessarily large number of API calls to +We made this choice because we found that the API did not support fetching multiple +specific books at once, which would have required an unnecessarily large number of API calls to achieve this without saving data - something the project description specified that we should not do. Another choice we have made is to include some external icons. All of these