Skip to content

Conversation

@AML441
Copy link
Collaborator

@AML441 AML441 commented Oct 31, 2025

Pull Request

Description

Gallery Component

  • Implemented frontend gallery component
  • Implemented tag component for each item's size, category, and quantity
  • Right now the gallery lists items that were created on the frontend
  • The black outline for the image is only there for sizing purposes and will be removed when the frontend is finalized

Search Bar Component

  • Implemented frontend search bar component
  • Integrated the search bar with the search function on the backend
  • By default, the gallery displays the items created on the frontend. But if you try to search something, because nothing is stored on firebase, nothing will show up.

Additional comments:

  • Next sprint, I will add in the specific icons and readjust the frontend responsiveness/proportions according to feedback while also working on next sprint tasks

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional changes)
  • Performance improvement
  • Other (please describe):

Related Issues (put task name here from notion)

Pre-Submission Checklist

Code Review Preparation

  • I have performed a self-review and reviewed the changed files
  • My code follows the project's style guidelines and passes linting
  • I have added comments in hard-to-understand areas and updated documentation

Testing (Ignore for the first PR)

  • I have added relevant tests and they pass locally
  • I have tested the changes manually, including edge cases

Code Quality & Security

  • I have formatted my code and removed debugging/unused code
  • No TypeScript errors or linting issues
  • No sensitive information committed (API keys, passwords, etc.)
  • User inputs are validated and sanitized

Dependencies & Deployment

  • Dependencies are documented and package-lock.json is updated if needed
  • Changes work with Firebase emulator and environment variables are documentedRetry

Screenshots (If it is a front end feature screenshot is required)

image

Additional Notes

@joelchem
Copy link
Collaborator

joelchem commented Nov 1, 2025

Since this is mostly frontend changes and there is still some styling that needs to be done I think I'm going to let this PR stay until the frontend is fully completed.

@joelchem joelchem closed this Dec 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants