Closed KamilaHasanbega closed 8 years ago
So this is not needed if #28 is merged? If you want to submit this one separately, please see my comments on #28 re: attachments removal.
@koto The CL is needed because it changes the messages controller while #28 changes the threads controller and its UI.
LGTM, safe to merge @kbsriram
Rebased via 3a17b4a
Addressed the #27 issue, of removing the wrong attachment.