search

CivicTechAtlanta / connector

A management tool for the brigade's projects
BSD 3-Clause "New" or "Revised" License
4 stars 12 forks source link

Person updates #13

Closed witbok closed 10 years ago

witbok commented 10 years ago

Adds a bunch of commits for the people objects in the app plus accompanying specs. Feel free to change anything!

witbok commented 10 years ago

@sb8244 -- I've made the suggested style updates and re-pushed. Let me know if I should change anything else.

sb8244 commented 10 years ago

Can you look into the current_user helper method? :+1: otherwise. I think you can just remove it and it will work.

witbok commented 10 years ago

@sb8244 I removed the helper method and nothing caught on fire. Is everything else good for merging?

sb8244 commented 10 years ago

Yes

sb8244 commented 10 years ago

@witbok once closed you can delete the branch remote for all of your PR.