Extracted out multiple methods of add method (#809)
Extracted out multiple methods of add method (#809)

* Extracted out multiple methods of add method

Detangled some of the logic that was tied up in the if statement by
extracting the logic determining the pusher_type. Created
`setup_allyship`, `remove_allyship_request`, and
`send_allyship_notifications` to help clarify what all is a part of the
add method within the allies_controller.

* codeclimate fix

* squash one bug two more appear :)

* correctly call the method!

julianguyen/ifme,julianguyen/ifme,cartothemax/ifme,julianguyen/ifme,julianguyen/ifme,cartothemax/ifme,cartothemax/ifme,cartothemax/ifme