-
Notifications
You must be signed in to change notification settings - Fork 4
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
Update README.md #123
Update README.md #123
Conversation
README.md
Outdated
- Dish Listings: Cooks can post dishes along with images, descriptions, categories, and allergy information. | ||
- Order Management: Users can browse dishes, place orders, and view their order history. | ||
- Cart: Users can add and remove items from their cart before finalizing orders. | ||
- Admin Dashboard: Cooks and Admin have extra access for managing dishes and orders. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In here maybe we can write in this way:
Admin Dashboard: Admin has extra access for users, dishes and orders.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
README.md
Outdated
|
||
- **User Authentication:** Implemented using Passport.js with Google strategy. | ||
- **Routes and Endpoints:** Defined clear routes and endpoints for user authentication, dish management, orders, and user profiles. | ||
- **Database Integration:** Utilized MongoDB to store user data, dishes, and order information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In here maybe we can add the cart.
- Database Integration: Utilized MongoDB to store user datas, dishes, carts and order informations.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
README.md
Outdated
- **User Authentication:** Implemented using Passport.js with Google strategy. | ||
- **Routes and Endpoints:** Defined clear routes and endpoints for user authentication, dish management, orders, and user profiles. | ||
- **Database Integration:** Utilized MongoDB to store user data, dishes, and order information. | ||
- **Admin Dashboard:** Created a separate route for Admin to manage dishes and orders. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In here maybe we can add the users.
Admin Dashboard: Created a separate route for Admin to manage users, dishes and orders.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Khaled,
Thank you for the creating READ.me file! It looks really good!
I add some comments for some lines.
In addition, I think, it would be good if we add packages that we used such as express, mongoose, jsonwebtoken, passport,
bcrypt, turkey-neighbourhoods, swagger-jsdoc, swagger-ui-express, express-async-handler, http-status-codes, jest, supertest, express-validator...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello Asli, thanks for ur feedback!
I did some changes, can u plz review it again
No description provided.