We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Very impressive work. The website is very easy to navigate and very beautifully done. There are just a few minor things which I have listed below:
The site logo incorrectly refers to /home instead of / and therefore clicking it leads the user to a blank page. https://github.com/Laila1222/meal-sharing/blob/16a514761bbdaa3876814618dce12b176cf17932/src/frontend/pages/home.js#L5
/home
/
The link in the top navigation menu for Become a host makes a GET request to the endpoint /add-meal. However, this endpoint seems to be configured to handle only POST requests. https://github.com/Laila1222/meal-sharing/blob/16a514761bbdaa3876814618dce12b176cf17932/src/backend/api/meals.js#L71
Become a host
/add-meal
Due to a missing " here, the user is taken to an incorrect URL. https://github.com/Laila1222/meal-sharing/blob/16a514761bbdaa3876814618dce12b176cf17932/src/frontend/pages/home.js#L45
"
You have commented here about this portion being still a work-in-process. Please let me know if you found this difficult to implement or could not make time for it. https://github.com/Laila1222/meal-sharing/blob/a67220b1210eb31580d74ebb9fcd01bab0f1f958/src/frontend/pages/reviewsAndBook.js#L133
Overall, extremely impressive work. Keep it up!
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Very impressive work. The website is very easy to navigate and very beautifully done. There are just a few minor things which I have listed below:
The site logo incorrectly refers to
/homeinstead of/and therefore clicking it leads the user to a blank page.https://github.com/Laila1222/meal-sharing/blob/16a514761bbdaa3876814618dce12b176cf17932/src/frontend/pages/home.js#L5
The link in the top navigation menu for
Become a hostmakes a GET request to the endpoint/add-meal. However, this endpoint seems to be configured to handle only POST requests.https://github.com/Laila1222/meal-sharing/blob/16a514761bbdaa3876814618dce12b176cf17932/src/backend/api/meals.js#L71
Due to a missing
"here, the user is taken to an incorrect URL.https://github.com/Laila1222/meal-sharing/blob/16a514761bbdaa3876814618dce12b176cf17932/src/frontend/pages/home.js#L45
You have commented here about this portion being still a work-in-process. Please let me know if you found this difficult to implement or could not make time for it.
https://github.com/Laila1222/meal-sharing/blob/a67220b1210eb31580d74ebb9fcd01bab0f1f958/src/frontend/pages/reviewsAndBook.js#L133
Overall, extremely impressive work. Keep it up!
The text was updated successfully, but these errors were encountered: