Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Bug | Fix Update Item #22

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Bug | Fix Update Item #22

wants to merge 2 commits into from

Conversation

saurookadook
Copy link
Contributor

@saurookadook saurookadook commented Feb 9, 2021 •
edited
Loading

Description

This PR fixes a bug with the Update Item button not working correctly in the ItemsList component.

Related Issues

#19

How to test

  • Make sure to have at least one Item
  • Click on "Update Item" from either Items list pages
  • Make changes to one or more fields in that Item and save
  • Check both JS console and Express logs for errors (there shouldn't be any 🙂 )

@saurookadook saurookadook added the bug Something isn't working label Feb 9, 2021
@saurookadook saurookadook added this to the Patch v1.03 milestone Feb 9, 2021
@saurookadook saurookadook self-assigned this Feb 9, 2021
@saurookadook saurookadook changed the title Change props.value to original._id Bug | Fix Update Item Feb 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@steve-louis steve-louis Awaiting requested review from steve-louis

@ovidio ovidio Awaiting requested review from ovidio

Assignees

@saurookadook saurookadook

Labels
bug Something isn't working
Projects
None yet
Milestone
Patch v1.03
Development

Successfully merging this pull request may close these issues.
1 participant
@saurookadook