search

tinode / ios

Tinodios: Tinode Messaging Client for iOS
Apache License 2.0
241 stars 104 forks source link

Image uploads #165

Closed or-else closed 3 years ago

or-else commented 3 years ago

I've implemented the same functionality as in web and Android for uploading large image and sending the URL. I've refactored the progress indicator: it's now a bar instead of a circle. It works OK for both attachments and images. The circle looked odd with images.

There is at least one unfixed bug (unrelated to uploads): when message sending fails and I delete the failed unsent message, the app tries to send {del} request to the server, fails, and that {del}-fail cycle is stuck forever.

or-else commented 3 years ago

It looks like the bug with deletion of failed messages is also present in Android. I'm going to try to fix them both before release.

aforge commented 3 years ago

It looks like the bug with deletion of failed messages is also present in Android. I'm going to try to fix them both before release.

Sounds good to me.