Skip to content

Shadi A.#3

Open
shmoonwalker wants to merge 7 commits intoHackYourAssignment:mainfrom
shmoonwalker:main
Open

Shadi A.#3
shmoonwalker wants to merge 7 commits intoHackYourAssignment:mainfrom
shmoonwalker:main

Conversation

@shmoonwalker
Copy link
Copy Markdown

finish week 9 assignment

@github-actions

This comment has been minimized.

@JQnetherlands JQnetherlands self-requested a review March 12, 2026 10:56
@JQnetherlands JQnetherlands self-assigned this Mar 12, 2026
Copy link
Copy Markdown

@JQnetherlands JQnetherlands left a comment

Choose a reason for hiding this comment

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

Hello,

I will start with the good news and after with the nice to haves, Congratulation you have passed task 1 and task 2 successfully, your code is well structured , it is easy to follow and most important to understand , keep the good job.

The only feedback I have to give you so you can grow as a stonger developer is the next one. In you task 2 services.js I can clearly see you do understand what the code is supposed to do, you show that you understand perfectly the documentation of URLSearchParams. On the other hand, the code could be much shorter if for example you use object destructuring how and where you could use it in your code?

I invite you to start to think how you can make your code doing the same with less, your code does it really well but could be shorter, that is something that come with time but it is good you start thinking about it from now on, again , congratulation you did an amazing job.

@JQnetherlands JQnetherlands added the Reviewed This assignment has been reivewed by a mentor and a feedback has been provided label Mar 17, 2026
@github-actions
Copy link
Copy Markdown

📝 HackYourFuture auto grade

Assignment Score: 100 / 100 ✅

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

Test Details
Vitest unit test results:

✅ post.sh: Script exists
✅ post.sh: Create user John Doe
✅ patch.sh: Script exists
✅ patch.sh: Correct the email address
✅ get.sh: Script exists
✅ get.sh: Retrieve user John Doe details
✅ delete.sh: Script exists
✅ delete.sh: Delete user John Doe
✅ get.sh: Verify user John Doe has been deleted
✅ Uses the correct base URL
✅ URL includes the nobelPrizeYear query parameter
✅ URL includes the nobelPrizeCategory query parameter
✅ URL includes the offset query parameter
✅ URL includes the limit query parameter
✅ Omits the nobelPrizeYear when set to "all"
✅ Omits the nobelPrizeCategory when set to "all"
✅ Includes the sort query parameter set to desc

Total passed: 17
Total failed: 0
Killed

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