martingaston / billy-search Goto Github PK

• Validate query on server

As someone who has searched for a book, I should get a list of books with each item in the list including:

• The book's author
• The book's title
• The book's publishing company
• A picture of the book.

Desired output:

Current output:

Line 35, refactor this into a proper error.

Can much of the API search be abstracted into its own class rather than as individual functions?

As someone who is searching for a book, each item in the list should have allow you to navigate to additional information about the book.

This information does not necessarily need to appear on a page within the application: it could link out to an external site with more information about that particular book.

• Wikipedia https://en.wikipedia.org/w/index.php?search=
• Google - https://www.google.com/q=
• Google Books - link within JSON

Code by intention, rather than translating intention with comments - e.g. line 15 on helpers. Consider amount of comments, variable and function names and think of ways to make code self-documenting.

Ensure consistency across files in how imports are handled

e.g. add tests and src subfolders, and look into Flask's recommended folder structure to make code more easily contextualised

Images are coming in over HTTP but the site is being served via HTTPS, hence the warning.

As someone visiting the site, I should be able to type in a query and display a list of books matching that query.

This is a catch-all for two separate things, consider the single responsibility principle here.

Index Page

• Search Bar

Results Page

• Top Bar Search Section
• Display Results
• Prev/Next buttons