implement_CSS_for_supply_request resolved #52
Conversation
| <HStack justify="flex-end" marginEnd={'5%'}> | ||
| <Button colorScheme="teal" variant="outline" width="130px"> | ||
| Cancel | ||
| <Link to="/BusinessDashboard"></Link> |
There was a problem hiding this comment.
rather than using a link, make an onclick for the button that uses a useNavigate to change the route
| <ModalFooter> | ||
| <Button colorScheme="blue" mr={3} onClick={onClose}> | ||
| Back to Home | ||
| <Link to={'/BusinessDonationTrackingForm'} /> |
There was a problem hiding this comment.
Rather than using a link, make the onclick for the button with a useNavigate that takes the user back to the home page
| Cancel | ||
| <Link to="/BusinessDashboard"></Link> | ||
| </Button> | ||
| <Button {...buttonStyles} disabled={isFormValid} width="130px" onClick={SubmitForm}> |
There was a problem hiding this comment.
the button is still clickable even when form is not valid. this is because the prop should be isDisabled not disabled.
| const newText = e.target.value; | ||
| setText(newText); | ||
| isFormFilled(); | ||
| setIsFormValid(checkedItems.some(value => value !== 0) && text.length > 0); |
There was a problem hiding this comment.
make this an or so that businesses can request supplies without a message or send a message without requesting supplies
| // Update the state with the new array | ||
| setCheckedItems(checkedItems); | ||
| isFormFilled(); | ||
| setIsFormValid(checkedItems.some(value => value !== 0) && text.length > 0); |
There was a problem hiding this comment.
make this an or so that businesses can request supplies without a message or send a message without requesting supplies
| @@ -33,38 +94,226 @@ const ContactUs = () => { | |||
| been_dismissed: false, | |||
| }; | |||
| const result = await backend.post('/notification/', updatedFormData); | |||
| console.log(result); | |||
There was a problem hiding this comment.
remove unnecessary console log
| const setMessage = text => { | ||
| const textCopy = text; | ||
| const checkedThingies = ['Stickers', 'Posters', 'Business Cards', 'Something', 'Joshua Lipton']; | ||
| const preMessage = `${name} is requesting: \n`; // business name... hard-coded right now | ||
| const checkedThingies = [ | ||
| 'DS Decals', | ||
| 'Rack Cards', | ||
| 'Donation Envelopes', | ||
| 'Business Cards', | ||
| 'Stickers', | ||
| 'Bin Decals', | ||
| 'DS Decals', | ||
| ]; | ||
| const preMessage = `Business is requesting: \n`; // business name... hard-coded right now | ||
| const updatedMessage = checkedThingies.filter((_, index) => checkedItems[index]).join('\n-'); | ||
| return `${preMessage}-${updatedMessage}\nNotes:\n${textCopy}`; | ||
| }; |
There was a problem hiding this comment.
FPH needs to know how many of each thing that businesses need so you should add a useState array to count the values of each supply the businesses are requesting
There was a problem hiding this comment.
In between the linebreak and the hyphen, there needs to be a number for the item requests.
| @@ -78,7 +327,7 @@ const ContactUs = () => { | |||
| <Text>Email</Text> | |||
| <Text>[email protected]</Text> | |||
| </Box> | |||
| </Box> | |||
| </Box> */} | |||
There was a problem hiding this comment.
delete unnecessary commented code
| // const handleCheck = (index, newValue) => { | ||
| // setCheckedItems(checkedItems => { | ||
| // const newArray = [...checkedItems]; | ||
| // newArray[index] = newValue; | ||
| // return newArray; | ||
| // }); | ||
| // }; |
|
|
||
| const ContactUs = () => { | ||
| const business_ID = 0; | ||
| const { backend } = useBackend(); | ||
| const [name, setName] = useState('business?'); // name of business | ||
| // const [name, setName] = useState('business?'); // name of business |
| const setMessage = text => { | ||
| const textCopy = text; | ||
| const checkedThingies = ['Stickers', 'Posters', 'Business Cards', 'Something', 'Joshua Lipton']; | ||
| const preMessage = `${name} is requesting: \n`; // business name... hard-coded right now | ||
| const checkedThingies = [ | ||
| 'DS Decals', | ||
| 'Rack Cards', | ||
| 'Donation Envelopes', | ||
| 'Business Cards', | ||
| 'Stickers', | ||
| 'Bin Decals', | ||
| 'DS Decals', | ||
| ]; | ||
| const preMessage = `Business is requesting: \n`; // business name... hard-coded right now | ||
| const updatedMessage = checkedThingies.filter((_, index) => checkedItems[index]).join('\n-'); | ||
| return `${preMessage}-${updatedMessage}\nNotes:\n${textCopy}`; | ||
| }; |
There was a problem hiding this comment.
Check the comment above ^^, make sure to add the number of items and insert it into the string.
Resolves #43