search

MidnightRiders / MemberPortal

Members-only application for the Midnight Riders
MIT License
2 stars 0 forks source link

Break up User.import #12

Closed bensaufley closed 7 years ago

bensaufley commented 7 years ago

That method is way too big and does way too much and also is not reusable in other contexts if I were to, say, want to just import one user that I have info for but don't want to create a CSV for it.

Also breaking it up makes it testable. Specs should be made when this is done.

bensaufley commented 7 years ago

Some work could still be done here but I'm going to close this because it's a lot better and not, I think, worth further effort at this point.