Skip to content

lab assignments solution#683

Open
Chinonye1 wants to merge 1 commit into
ironhack-labs:masterfrom
Chinonye1:master
Open

lab assignments solution#683
Chinonye1 wants to merge 1 commit into
ironhack-labs:masterfrom
Chinonye1:master

Conversation

@Chinonye1
Copy link
Copy Markdown

No description provided.

Copilot AI review requested due to automatic review settings May 9, 2026 07:52
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates src/books.js to implement parts of the “JavaScript Book Objects” lab by populating the booksArray data and adding logic for several iterations.

Changes:

  • Replaced the placeholder booksArray with four concrete book objects (including nested details).
  • Implemented getBookDetails(book) and added top-level logic to delete details.language and compute readingTime.
  • Renamed the provided dictionary constant to dictionaryExample (aligning with the README), but left the Bonus functions incomplete.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/books.js
Comment on lines +57 to 60
getBookDetails(booksArray)



Comment thread src/books.js
Comment on lines +65 to +69
console.log(booksArray)
return element


})
Comment thread src/books.js
Comment on lines +80 to +82
console.log(booksArray)
return goods
})
Comment thread src/books.js
Comment on lines +110 to +112
function booksByAuthor() {}
// Your code here:

}

Comment thread src/books.js
Comment on lines +50 to +53
function getBookDetails(details) {
// Your code here:

return (`${details.title} - ${details.author} - ${details.pages} pages`)
Comment thread src/books.js
Comment on lines +110 to 115
function booksByAuthor() {}
// Your code here:

}




Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants