Conversation
Name, Description, Price, Category, Image Express Route to GET all Items Front-end View for all Items
…-App into individual-items
Individual items
Did some small changes to the visuals
…g item in single view
Bhavik branch
…works and refreshes
added refresh to the back button
…nd-orders creating Users and order files
updated css and app.js
…into backend-testing
Newjosh dev
Backend testing
…run both frontend and backend tests
Hopefully last Attempt to Merge
LaurieMV
left a comment
LaurieMV
left a comment
There was a problem hiding this comment.
Great job, team! You completed all tiers plus lots of bonus. You did a nice job with your design and wireframe. The app has consistent styling. You provided sort, a shopping cart and orders. You wrote automatic tests for both the client and the server, with especially good coverage of the server. Your README is sufficient but slightly out of date.
| <> | ||
| <h1>Inventory App</h1> | ||
| {/* Render the items */} | ||
| {/* 🧾 App Title */} |
There was a problem hiding this comment.
There is a lot going on here. Consider refactoring into new components, e.g. for the Item List and the Cart.
| "version": "3.0.0", | ||
| "scripts": { | ||
| "test": "jest", | ||
| "test:server": "jest --config=jest.server.config.js --runInBand", |
There was a problem hiding this comment.
Glad to see you were able to do testing! The server tests have almost complete coverage, way to go.
| category: "", | ||
| }) | ||
| } catch(err) { | ||
| throw new Error(err); |
There was a problem hiding this comment.
what will the user see when this error occurs?
| width="20%" | ||
| height="20%" | ||
| /> | ||
| <h3 className="price">Price: ${item.price}</h3> |
There was a problem hiding this comment.
If price is 22.30, will this show 22.3?
Look into Intl.NumberFormat for displaying your price using best practices, here and in the forms
| @@ -1,5 +1,19 @@ | |||
| const Item = require("./Item"); | |||
| const User = require("./User"); | |||
There was a problem hiding this comment.
Great additions to ordering functionality
Should a Cart create an Order?
|
|
||
| You are a team of developers for an e-commerce company. The Engineering team is rebuilding their inventory tracking app from the ground up. Your team has been tasked with creating a Full-Sack (front and back end) RESTful CRUD application to track the items. | ||
| # Description of the App | ||
| Our Inventory App provides users with comprehensive inventory management capabilities, including the ability to view, update, and delete specific items, as well as seamlessly add new inventory entries. The application also incorporates an advanced search feature, enabling efficient retrieval of items based on their names to streamline inventory operations. |
There was a problem hiding this comment.
You have shopping cart as well
7 participants
Don’t close or merge this pull request, unless you’re instructed to do so by your teacher.