Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quoting on talk app android doesnt work well #4101

Open
GretaD opened this issue Aug 20, 2024 · 6 comments
Open

Quoting on talk app android doesnt work well #4101

GretaD opened this issue Aug 20, 2024 · 6 comments
Assignees
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) bug Something isn't working feature: 📵 offline support
Milestone

Comments

@GretaD
Copy link

GretaD commented Aug 20, 2024

How to use GitHub

  • Please use the 👍 reaction to show that you are affected by the same issue.
  • Please don't comment if you have no relevant information to add. It's just extra noise for everyone subscribed to this issue.
  • Subscribe to receive notifications on status change and new comments.

Steps to reproduce

  1. Open Talk app on an android phone
  2. Open an existing conversation and quote a message you already send and write something as reply
  3. See the quote not showing up

Expected behaviour

I see the message i quoted

Actual behaviour

Quoting is empty
talkapp

Talk app

Talk app version: (see apps administration page: /index.php/settings/apps)

Custom Signaling server configured: yes/no and version (see Talk administration settings: /index.php/settings/admin/talk#signaling_server)

Custom TURN server configured: yes/no (see Talk administration settings: /index.php/settings/admin/talk#turn_server)

Custom STUN server configured: yes/no (see Talk administration settings: /index.php/settings/admin/talk#stun_server)

Browser

Microphone available: yes/no

Camera available: yes/no

Operating system: Windows/Ubuntu/Mac/...

Browser name: Firefox/Chrome/Safari/...

Browser version: 124/125/...

Browser log

``` Insert your browser log here, this could for example include: a) The javascript console log b) The network log c) ... ```

Server configuration

Operating system: Ubuntu/RedHat/...

Web server: Apache/Nginx

Database: MySQL/Maria/SQLite/PostgreSQL

PHP version: 8.1/8.2/8.3

Nextcloud Version: (see administration page)

List of activated apps:

If you have access to your command line run e.g.:
sudo -u www-data php occ app:list
from within your server installation folder

Nextcloud configuration:

If you have access to your command line run e.g.:
sudo -u www-data php occ config:list system
from within your Nextcloud installation folder

Server log (data/nextcloud.log)

Insert your server log here
@GretaD GretaD added 0. Needs triage bug Something isn't working labels Aug 20, 2024
@nickvergessen nickvergessen transferred this issue from nextcloud/spreed Aug 20, 2024
@nickvergessen nickvergessen added 1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) and removed 0. Needs triage labels Aug 20, 2024
@GretaD
Copy link
Author

GretaD commented Aug 20, 2024

I was chatting with @ChristophWurst, i would be curious if Christoph can see the quoting on his phone. Because all looks good on web.

@AndyScherzinger
Copy link
Member

@rapterjet2004 I also experienced this today. The message showed up when I navigated back to the list of conversations and back into the same conversation. So there seems to be some rendering issue. On top it also had issues with rendering the content correctly (link has been too "long") however I can't share a screenshot of that conversation.

@rapterjet2004
Copy link
Contributor

Not sure if this can be an easy fix, because since switching to offline first we don't keep the parent message directly in storage, but instead save it's id parentMessageId: Long? so it can be looked up in the database later. However, if the parent message itself it not saved ( for example If it's really old ) then the UI only shows an empty String. However, when you scroll up, the message is then saved to storage but this doesn't immediately refresh the adapter unless you reenter the conversation.

I think what I can do is add some progress indicators to make the UI more user friendly. I'll talk to @mahibi about it when he comes back.

Not sure about the long link though, I was unable to recreate that bug. I tested it with this and it seems to render fine. Although I do notice some flickering, probably has something to do with the refresh logic. Will have to fix that.

Screenshot 2024-08-23 at 11 47 54 AM

@SystemKeeper
Copy link

SystemKeeper commented Aug 23, 2024

Not sure if this works for your offline implementation, but we update/store a parent message when we receive a message: https://github.com/nextcloud/talk-ios/blob/df94d83f60c30d256ecb1ba1e5e50f40e16491c0/NextcloudTalk/NCChatController.m#L130 so we always have the parent in store

@mahibi mahibi added this to the 20.0.0 milestone Sep 2, 2024
@mahibi
Copy link
Collaborator

mahibi commented Sep 6, 2024

i can't reproduce the empty quoting. @GretaD @AndyScherzinger does it happen always for you or sometimes?

@AndyScherzinger
Copy link
Member

Doesn't happen all the time. To me only happened in the beginning. By now I am running latest master and haven't experienced it again (maybe due to @rapterjet2004 comment).

@mahibi mahibi modified the milestones: 20.0.0, 20.0.1, 20.0.2 Sep 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1. to develop Accepted and waiting to be taken care of (should be only set by nextcloud employees) bug Something isn't working feature: 📵 offline support
Projects
None yet
Development

No branches or pull requests

6 participants