Business Donation History Resolved #36
Conversation
| <Td>{item.reporter}</Td> | ||
| <Td>{item.food_bank_donation}</Td> | ||
| <Td>{item.date}</Td> |
There was a problem hiding this comment.
iterate through the columns rather than explicitly accessing each column. see BusinessTable for an example.
| const { backend } = useBackend(); | ||
| const [data, setData] = useState([]); | ||
| const [selectedDonationId, setSelectedDonationId] = useState(null); | ||
| const TABLE_HEADERS = ['reporter', 'food_bank_donation', 'date', 'action']; |
There was a problem hiding this comment.
nit: format cleanly since these are the values that appear to the user
| // ViewBusiness.PropTypes = { | ||
| // id: PropTypes.number.isRequired, | ||
| // } |
There was a problem hiding this comment.
delete commented code if you're not using it
| console.log(`Fetching donation details for ID: ${id}`); | ||
| const response = await backend.get(`/donation/${id}`); | ||
| setData(response.data); | ||
| console.log('Response received:', response); |
There was a problem hiding this comment.
remove unnecessary console.logs
| try { | ||
| setSelectedDonationId(id); | ||
| const response = await backend.get(`/donation/${id}`); | ||
| console.log(response.data); |
There was a problem hiding this comment.
remove unnecessary console.log
| } | ||
| }; | ||
| getData(); | ||
| }, []); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'backend'. Either include it or remove the dependency array.
| if (id) { | ||
| fetchDonation(); | ||
| } | ||
| }, [id]); |
There was a problem hiding this comment.
React Hook useEffect has a missing dependency: 'fetchDonation'. Either include it or remove the dependency array.
resolves #33