Closed liam-strand closed 3 weeks ago
inventory-viewing
page has hardcoded dummy data/ is not hooked up to databaseitem-detail-test
page is a test pageimage-capture-and-upload-component
speaking in demonic tongues when u try to take a picitem-upload-page-test
has hardcoded data-Naming of folders should be universal (all camel case or all with en-dashes) -Running the dev server after pulling from main doesn't work correctly -Search Item API (searching part) does not function on the inventory viewing page -Inventory viewing tags are hard coded (dummy data, replace this with something else) -Error when trying to take picture before the camera loads -Error 500 when trying to add an item on item-upload-page-test -When camera access is denied, there is no indication for the user to allow camera settings for the site
src/app/item/[id]/page.tsx
line 28: make a proper not found pagesrc/app/item/[id]/page.tsx
line 41: load images properlysearch-item
route is dirtyyyyyyy bc it has the backend implementation commented out-Close button way too small when added an iframe from a youtube link
neon()
. Perhaps an environment variable has not been set?-No reply functionality for comments on an item
(its not our fault)
- [ ] back to inventory page button doesn't work
Also, broken for individual item summary pages
-No indication of "no results found" for an item if the filters don't match
- [ ] Expected behavior??
I'm going to go with yes, that is expected, @brandondionisio.
Please comment below with any issues you find in the site
These include...
console.log
sPlease be specific and concise in your description. Here are some examples:
Notice that those are all checkboxes. Please do that sort of list so that we can easily convert them into GitHub issues. You can do a checkbox list by hitting the "task list" button in the editor or by typing
- [ ] yourtexthere
in the text field (yay markdown).Additionally, please make sure you are testing on the
main
branch. Does this mean you might catch issues that are already fixed in another branch waiting to be merged? Yes, but it makes sure that we are all testing against the same baseline.