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

65 Update Status Form #72

Merged
merged 11 commits into from
Nov 18, 2024
Merged

65 Update Status Form #72

merged 11 commits into from
Nov 18, 2024

Conversation

truffer11
Copy link
Collaborator

Pull Request Template

**Fixes #65 **

What was changed?

  • Made a new form that can be used to update the status of donated items in the backend.

Why was it changed?

  • Allows for your average user to update the status of the item from the frontend.

How was it changed?

  • Added StatusUpdate.tsx. This generates a form that allows a user to input a new status from a dropdown and select the date of the update. There will be functionality to add photos in the future, but that is not slated for this sprint.
  • Updated routing to make localhost:3000/donatedItem/status/(id#) to direct to the new form.
  • There is an attempt to use PUT to update the item, however that functionality is not working as expected. Will be fixed by merge day.

Screenshots that show the changes (if applicable):

  • Before:
    • The void
  • After:
    • Screenshot 2024-11-11 153355

@truffer11 truffer11 linked an issue Nov 11, 2024 that may be closed by this pull request
4 tasks
server/package.json Outdated Show resolved Hide resolved
Copy link
Collaborator

@miabu-pashh miabu-pashh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Stash this file after commiting in local , and send only the file which has been changed.
  • Change the Content-Type': 'application/json' is more appropriate.
  • Remove unwanted comments
  • Remove the unwanted dependencies dependencies are not need for this update

client-app/src/Components/StatusDisplayPage.js Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
server/package.json Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
client-app/src/Components/StatusUpdate.tsx Outdated Show resolved Hide resolved
@Anjali0407-git Anjali0407-git added this to the Sprint 5 milestone Nov 17, 2024
@Anjali0407-git Anjali0407-git merged commit 6d83a75 into main Nov 18, 2024
3 checks passed
@Anjali0407-git Anjali0407-git deleted the 65-update-status branch November 18, 2024 21:19
@truffer11 truffer11 restored the 65-update-status branch November 19, 2024 00:26
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.

Implement Frontend Form for Adding New Status at /donatedItem/status/:id
3 participants