search

codeforamerica / follow-all

Follow all the members of a Twitter list with a single click
http://follow-all.herokuapp.com/
BSD 3-Clause "New" or "Revised" License
81 stars 31 forks source link

Fixed error when the currently logged on user is in a list #27

Closed kacole2 closed 8 years ago

kacole2 commented 8 years ago

When a user tried to a follow a list of people and the user is in that list, it would error out. There is an error checking process on the array of twitter IDs. The client ID is gathered and stored as a variable and then a loop is ran against the array of new people. If the IDs are a match, then it's removed from the array.

This is requested from #22

Signed-off-by: Kendrick Coleman kendrickcoleman@gmail.com

kacole2 commented 8 years ago

@sferik i can't get the test to pass. I've fixed all the offenses. I'm not sure what I need to do in the follows_controller_test.rb to make it pass with the 3 lines of code I added.

thanks!

kacole2 commented 8 years ago

eureka!