Skip to content

MohammedA#17

Open
mohammedalfakih-dev wants to merge 1 commit intoHackYourAssignment:mainfrom
mohammedalfakih-dev:main
Open

MohammedA#17
mohammedalfakih-dev wants to merge 1 commit intoHackYourAssignment:mainfrom
mohammedalfakih-dev:main

Conversation

@mohammedalfakih-dev
Copy link

No description provided.

@github-actions
Copy link

📝 HackYourFuture auto grade

Assignment Score: 0 / 100 ✅

Status: ✅ Passed
Minimum score to pass: 0
🧪 The auto grade is experimental and still being improved

Test Details

@mohammedalfakih-dev mohammedalfakih-dev changed the title Finish reading list app MohammedA Feb 18, 2026
@hacdias hacdias self-assigned this Feb 22, 2026
Copy link

@hacdias hacdias left a comment

Choose a reason for hiding this comment

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

Hey! Good work overall. Not a lot of comments.

For the next assignments, you can remove the TODO comments from the code also.

Have a nice week!

function getUnreadBooks() {
export function getUnreadBooks() {
// TODO: Implement this function using filter()
return loadBooks().filter((book) => book.read === false);
Copy link

Choose a reason for hiding this comment

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

Note that this will only match if the casing matches. I would've put everything lowercase for the check.

@hacdias hacdias added Reviewed This assignment has been reivewed by a mentor and a feedback has been provided and removed To review labels Feb 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Reviewed This assignment has been reivewed by a mentor and a feedback has been provided

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants