You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
You have a lot of functions in App.js. Might be good to move these to another file and import them. Cleans up App.js and makes these available everywhere if needed.
You need to do some linting! I see a lot of linter errors/warnings.
Consider destructuring props in InventoryItem. Consider
Great work on the product list looks like you solved all of the problems. It all works well. Here are a few suggestions.
Make a folder for components an use a sub-folder for components that rely on other files.
src
-- App
--- App.css
--- App.js
-- InventoryItem.js
-- DisplayAllBtn.js
-- CategoryBtn.js
You have a lot of functions in App.js. Might be good to move these to another file and import them. Cleans up App.js and makes these available everywhere if needed.
You need to do some linting! I see a lot of linter errors/warnings.
Consider destructuring props in InventoryItem. Consider
vs
I feel like DisplayAllBtn and CategoryBtn are so close you can make a single class. Use a prop to for the category name and give the all button null.
The text was updated successfully, but these errors were encountered: