clear composer and enable send button before sending messages

WideChat / Rocket.Chat.Android

Rocket.Chat Native Android Application

https://rocket.chat

MIT License

12 stars 6 forks source link

clear composer and enable send button before sending messages #560

Closed bizzbyster closed 5 years ago

bizzbyster commented 5 years ago

Closes #547

I simply clear the draft message before calling sendMessage/updateMessage, which can be subject to a 15 second timeout delay.

bizzbyster commented 5 years ago

I fixed an issue with this fix. Not seeing the BulkWriteError now.

kb0304 commented 5 years ago

@bizzbyster Can you please explain what was causing the error and how the fix fixes the BulkWriteError?

bizzbyster commented 5 years ago

I should have been more clear. I didn't necessarily fix BulkWriteError but I am not able to reproduce it now. My fix removed the enable of the button in "finally".

//view.enableSendMessageButton()

I think BulkWriteError is still a thing. But I also saw it without this change so I don't know that it is related to this PR.