neilsarkar
commented
6 years ago Dope. Yeah I think hilariously the only real API change is removing the match middleware on the messages endpoint
Closed benjaminben closed 6 years ago
neilsarkar
commented
6 years ago Dope. Yeah I think hilariously the only real API change is removing the match middleware on the messages endpoint
First of all I don't expect you to approve/merge this -- just thought it'd be good to start the PR. There's a ton of really obvious shit to do in terms of cleaning up code and organizing stuff, not to mention styling navigation buttons etc. But I figure you'll wanna do some of the major API tasks before we submit what I've started here (the biggest that comes to mind is basically getting rid of matches entirely, just letting people create conversations from
Recs). We'll need to actually write solid blocking/reporting features too lol, though I guess that can really wait until the launch version.Overall I feel really good about the direction of this; you were 100000% right about removing the tab bar, the app feels more premium now even with the donky square/flat navigation buttons up top. Let's discuss remaining steps to make all this merge-ready.