Skip to content
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

finished add item #120

Merged
merged 1 commit into from
May 26, 2024
Merged

finished add item #120

merged 1 commit into from
May 26, 2024

Conversation

zmattes04
Copy link
Collaborator

Developer: Zach Mattes

Closes #73

Pull Request Summary

Added a functioning add item component that works with frontend and backend
{Describe the purpose of your pull request}

Modifications

created NewItemOptions.tsx
And edited basket.tsx to work with the button

{list out the files created/modified and a brief description of what was changed}

Testing Considerations

tested with frontend and checked with backend. The baskets page still hides private items from every being shown but i think that is not part of my add items

{list out what you have tested and what the reviewer should verify}

Pull Request Checklist

  • Code is neat, readable, and works
  • Comments are appropriate
  • The commit messages follows our guidelines
  • The developer name is specified
  • The summary is completed
  • Assign reviewers

Screenshots/Screencast

{put screenshots of your change, or even better a screencast displaying the functionality}

@zmattes04 zmattes04 linked an issue May 26, 2024 that may be closed by this pull request
@zmattes04 zmattes04 requested a review from SilveerDusk May 26, 2024 19:10
@SilveerDusk
Copy link
Collaborator

Looks good to me!

@SilveerDusk SilveerDusk merged commit 967a93c into main May 26, 2024
3 checks passed
@jhagendoornCP jhagendoornCP deleted the 73-create-the-add-item-component branch June 4, 2024 21:10
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.

Create the Add Item Component
2 participants