Skip to content

docs: add project choices and reasoning #37

Merged
merged 2 commits into from
Sep 18, 2026
+12 −0
Merged

docs: add project choices and reasoning #37

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
bb2d3a5
docs: add project choices and reasoning
hestro Sep 18, 2026
6b5de01
docs: fix API limitation reasoning
hestro Sep 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down