Make the alert popup on delete of client a MUI modal component (ClientListTableRow.jsx)

[xscottxbrownx]

Issue:

When clicking the delete icon in the table of clients, an alert message pops up to confirm. Need to use an MUI modal and styling to match rest of app instead.

---

Direction:

Take inspiration from our other MUI modals to have similar styling for this.
LogoutModal, AddClientModal (which isn't 100% complete yet), and even InactivityMessage are all closely styled and fit within the app theme.

[ajay2827]

@xscottxbrownx I am interested in this, can you assign this to me?

[xscottxbrownx]

@xscottxbrownx I am interested in this, can you assign this to me?

Have you read this repo's readme's on contributing and such?

When creating new branch from Development for this, make sure to start branch name with issue-223/ followed by whatever you want to call branch (something like: "delete-client-modal")

[ajay2827]

@xscottxbrownx I have a made delete client modal and it works fine, I have attached a screenshot of it, could you review it, please?

[xscottxbrownx]

@xscottxbrownx I have a made delete client modal and it works fine, I have attached a screenshot of it, could you review it, please?

Well, all I can do right now is see what it looks like from your screenshot.

Can you make a pull request, so I can test on local machine and look at the code?

---

And in the future, could you make sure you get assigned to the issue before proceeding?
You didn't get assigned because you never responded to my questions.

If an issue isn't assigned, other developers will think it's free to be worked on and it will lead to somebody wasting their time or merge conflicts.

[ajay2827]

@xscottxbrownx I have a made delete client modal and it works fine, I have attached a screenshot of it, could you review it, please?

Well, all I can do right now is see what it looks like from your screenshot.

Can you make a pull request, so I can test on local machine and look at the code?

And in the future, could you make sure you get assigned to the issue before proceeding? You didn't get assigned because you never responded to my questions.

If an issue isn't assigned, other developers will think it's free to be worked on and it will lead to somebody wasting their time or merge conflicts.

As a beginner to open source, I didn't know about it, but in the future, I will take care of it, thank you @xscottxbrownx for helping me.

[ajay2827]

@xscottxbrownx I have a made delete client modal and it works fine, I have attached a screenshot of it, could you review it, please?

Well, all I can do right now is see what it looks like from your screenshot.

Can you make a pull request, so I can test on local machine and look at the code?

And in the future, could you make sure you get assigned to the issue before proceeding? You didn't get assigned because you never responded to my questions.

If an issue isn't assigned, other developers will think it's free to be worked on and it will lead to somebody wasting their time or merge conflicts.

Now i made a pull request with the title Created DeleteClientModal, could you review, please

[xscottxbrownx]

Now i made a pull request with the title Created DeleteClientModal, could you review, please

Thank you.

Also just another thing to help with GitHub use... you can link to an Issue or PR by using it's number.

So in this case you could add #257 and it would provide a link to the PR you created. As soon as you type the # symbol it should popup a list for you to select from. Just another tip.

[andycwilliams]

We may want to reassign this to @xscottxbrownx since he is now taking over responsibility.

[leekahung]

Assignment switched to @xscottxbrownx