1785 Add export button and notification dialogs

[kdow]

Fixes #1785

• Up to date with main branch
• Branch name follows guidelines
• All PR Status checks are successful
• Peer reviewed and approved

Any questions? See the getting started guide

[traycn]

Hey @kdow I pulled the branch and it's running great on my end, just came across a minor issue with MUI.

For functionality, I tested the export feature with multiple requests with each request containing a archive folder of 2 files: NeighborhoodData.csv and ServiceRequestCount.csv. It's working great!

On the UI side of things, I get a console error for having the onHandler function exec in the Button element:

There is an IconButton element in MUI that the error suggests we use. Links to the doc and examples below. What is your bandwidth for changing the Button element to IconButton as a potential fix for this console error?

IconButton Doc & Examples:
https://mui.com/material-ui/api/icon-button/
https://mui.com/material-ui/api/icon-button/

PS appreciate all the time put into this!

[kdow]

On the UI side of things, I get a console error for having the onHandler function exec in the Button element:

There is an IconButton element in MUI that the error suggests we use. Links to the doc and examples below. What is your bandwidth for changing the Button element to IconButton as a potential fix for this console error?

IconButton Doc & Examples: https://mui.com/material-ui/api/icon-button/

Thanks for catching that. It looks like it was referencing an IconButton component I created, and I just needed to move the onClick handler into it from the child Icon component. I've tested and it looks like the error went away. I've just updated the PR with the fix.

[traycn]

Thanks! Pulled the changes and came across another small issue. The functionality works great, just another MUI issue from a type check.

Expecting that it's being thrown at the following:
components/common/ChipList/StyledChip.jsx --- line 84
label: PropTypes.string.isRequired,

[kdow]

Fixed!

[traycn]

LGTM thank you!

[ryanfchase]

Minor changes requested, but overall this is great work. Ty @kdow 👍

[ryanfchase]

New changes are looking good, approved. Merge when ready 👍