ElixirCLE / pairmotron

Programming Pair Pairifier
https://pairmotron.herokuapp.com
5 stars 4 forks source link

Add ability to delete group invitations #138

Closed mbramson closed 7 years ago

mbramson commented 7 years ago

Adds controller actions and links on the UI to be able to delete invitations if you are the owner of the group associated with the invitation or the user associated with the invitation.

Closes #102

ericworkman commented 7 years ago

Code looks good to me. I think its slightly confusing to a user when their request gets denied, as there's no feedback to the requesting user or the group owner in the reverse case.

mbramson commented 7 years ago

Agreed. As discussed, we would need some sort of notifications system, and that's outside of the scope of this card. The user just not being in the group will have to do for now.