Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Books API #519

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,4 +10,4 @@ Describe how you approached to problem, and what tools and techniques you used t

## View it live

Every project should be deployed somewhere. Be sure to include the link to the deployed project so that the viewer can click around and see what it's all about.
https://project-express-api-17pi.onrender.com
1 change: 1 addition & 0 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@
"@babel/preset-env": "^7.16.11",
"cors": "^2.8.5",
"express": "^4.17.3",
"express-list-endpoints": "^7.1.1",
"nodemon": "^3.0.1"
}
}
65 changes: 55 additions & 10 deletions server.js
Original file line number Diff line number Diff line change
@@ -1,30 +1,75 @@
import express from "express";
import cors from "cors";

// If you're using one of our datasets, uncomment the appropriate import below
// to get started!
// import avocadoSalesData from "./data/avocado-sales.json";
// import booksData from "./data/books.json";
// import goldenGlobesData from "./data/golden-globes.json";
// import netflixData from "./data/netflix-titles.json";
// import topMusicData from "./data/top-music.json";
import booksData from "./data/books.json";

// Defines the port the app will run on. Defaults to 8080, but can be overridden
// when starting the server. Example command to overwrite PORT env variable value:
// PORT=9000 npm start
const port = process.env.PORT || 8080;
const app = express();
const listEndpoints = require("express-list-endpoints");

// Add middlewares to enable cors and json body parsing
app.use(cors());
app.use(express.json());

// Start defining your routes here
app.get("/", (req, res) => {
res.send("Hello Technigo!");
const endpoints = listEndpoints(app);
res.json({
message: "Welcome to the Book API!",
endpoints: endpoints
});
});

// Popular books with rating >4
app.get("/books/popular", (req, res) => {
const popularBooks = booksData.filter(book => book.average_rating && parseFloat(book.average_rating) > +4)
res.status(200).json(popularBooks)
});
Comment on lines +26 to 29
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This could be a query param: ?popular=true


// author and title
app.get("/books", (req, res) => {
const { title, author } = req.query;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! ⭐


let filteredBooks = booksData;

if (title) {
filteredBooks = filteredBooks.filter(book =>
book.title.toLowerCase().includes(title.toLowerCase())
);
}

if (author) {
filteredBooks = filteredBooks.filter(book => {
if (!book.authors) return false;

const authorsArray = book.authors.split("-").map(author => author.trim().toLowerCase());
return authorsArray.some(auth => auth.includes(author.toLowerCase()));
});
}

if (filteredBooks.length === 0) {
return res.status(404).json({ error: "No books found matching the criteria" });
}

res.status(200).json(filteredBooks);
});

// ISBN
app.get("/books/:isbn", (req, res) => {
const isbn = req.params.isbn
const book = booksData.find(book => book.isbn === +isbn)
res.json(book); {
if (book) {
res.status(200).json(book)
} else {
res.status(404).send("no book found with that ISBN")
}
}
})
Comment on lines +60 to +70
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


// Start the server
app.listen(port, () => {
console.log(`Server running on http://localhost:${port}`);
});
});