search

ncss-2014-group-3 / word-by-word

Group 3's word at a time social network
7 stars 4 forks source link

Misc Improvements #46

Closed nhardy closed 10 years ago

nhardy commented 10 years ago

Including previous commits (other pull request(s)).

nhardy commented 10 years ago

Think I'll wait for @plausibility's approval on this one.

nhardy commented 10 years ago

Had a chance to look at this one, @Mause or @plausibility?

sysr-q commented 10 years ago

Aside from those small things I commented on, it seems alright. I haven't tested it, however, so there may well be corner cases that break.

nhardy commented 10 years ago

I've tested it reasonably thoroughly prior to the commits today, but I don't think they've changed anything drastic so I think this should be Ok to merge now unless there's anything else you think I should change?

nhardy commented 10 years ago

Guess I'll go ahead and merge this in later if there are no more comments.