Add Delete User

[calebbourg]

Description

This PR adds the ability for a Coach to delete a Coachee.

Constraints:

• The current user can only delete other users that are direct Coachees
• The current user cannot delete themself

Changes

• Adds delete button to members page

Screenshots / Videos Showing UI Changes (if applicable)

Testing Strategy

Concerns

describe any concerns that might be worth mentioning or discussing

[jhodapp]

Looking great, works well. Some minor inline changes and I'll be happy to approve.

[jhodapp]

@calebbourg reminder as discussed in Slack: The Member page has 2 titles and only needs one on that page, and it should be an <h3> element. Would you mind making that update in this PR too since it’s such a minor change? Make sure it matches the title in style on the Dashboard page.

[calebbourg]

@jhodapp Thank you for the review! I think I've addressed everything. Including removing the extra title on the members page

[jhodapp]

@calebbourg The "Add Member" button is now on the left side of the screen instead of the right, which it should be on the right to be consistent with other pages.

[calebbourg]

@jhodapp How does this look dfa4cb3

[jhodapp]

@jhodapp How does this look dfa4cb3

@calebbourg Looks great, are you sure this is the right commit? It's from 2 days ago, I don't see a pushed commit that changes the button to be right justified.

[calebbourg]

@jhodapp I believe it is. I moved the "Members" text into the actual component and it placed the button to that side. I chose that approach to couple the both the text and the button with the list itself. Let me know what you think about that.

[jhodapp]

@jhodapp I believe it is. I moved the "Members" text into the actual component and it placed the button to that side. I chose that approach to couple the both the text and the button with the list itself. Let me know what you think about that.

I think that's a fine approach for our needs right now on the page, and we an change it in the future easily if necessary.