mattermost / mattermost-mobile

Next generation iOS and Android apps for Mattermost in React Native
https://about.mattermost.com/
Apache License 2.0
2.2k stars 1.33k forks source link

Unable to see more then 20 direct message users #7072

Closed CptLolliPants closed 1 year ago

CptLolliPants commented 1 year ago

Summary

The new Android app shows no more then 20 available direct message conversations, all others are unreachable

Environment Information

Steps to reproduce

Open Mattermost Open any team that has more then 20 direct message conversations remembered

Expected behavior

The list of direct messages per team show scroll until no more direct message conversations are available

Observed behavior (that appears unintentional)

Default startup screen shows the channels and direct messages. The Direct messages show no more then 20 entries before scrolling is stopped.

Possible fixes

Looks like a code issue. Not sure where this can be solved, sorry :S

amyblais commented 1 year ago

@CptLolliPants How many DMs have you configured to show on webapp/desktop app?

Screenshot at Feb 01 09-24-09
CptLolliPants commented 1 year ago

Apparently that was set to 20. Setting it to all solved the issue (although one can ask why this is even a thing and especially why it is not obvious in the Android App). The most annoying thing was mostly that I got a badge for unread messages and when you tapped that, it would go to this list of 20 where that specific user was not present in and it did not open that direct message conversation.

Thanks for now at least

On Wed, Feb 1, 2023 at 3:25 PM Amy Blais @.***> wrote:

@CptLolliPants https://github.com/CptLolliPants How many DMs have you configured to show on webapp/desktop app?

[image: Screenshot at Feb 01 09-24-09] https://user-images.githubusercontent.com/29708087/216069359-3da8f2d2-1236-44aa-8dea-b20b8f728b8a.png

— Reply to this email directly, view it on GitHub https://github.com/mattermost/mattermost-mobile/issues/7072#issuecomment-1412147227, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIY35KYZLKJ7RVE2G5YPURDWVJW5VANCNFSM6AAAAAAUNYOWLI . You are receiving this because you were mentioned.Message ID: @.***>

-- Vriendelijke groet, Sjoerd de Jong

amyblais commented 1 year ago

@enahum @matthewbirtch Is this currently expected?

matthewbirtch commented 1 year ago
  1. The number of DMs that show on mobile is directly related to your settings in the web app. Settings > Sidebar > Number of direct messages to show (or you can also change this through the menu that @amyblais has shown in her screenshot above). Currently, this setting is not available on the mobile app, but that's something we can consider adding in the future.
  2. If you have a direct message that has a mention badge on it, it should certainly be prioritized in your list of Direct Messages. @CptLolliPants you've mentioned that the specific DM with a mention did not surface in your list of 20 DMs. Is that correct? If so, this may be a bug.

@enahum anything to add?

CptLolliPants commented 1 year ago

Number 2 is exactly what happened. I had my list ordered alphabetically and the user did not appear when opening the badge.

On Fri, 3 Feb 2023, 17:11 Matthew Birtch, @.***> wrote:

  1. The number of DMs that show on mobile is directly related to your settings in the web app. Settings > Sidebar > Number of direct messages to show (or you can also change this through the menu that @amyblais https://github.com/amyblais has shown in her screenshot above). Currently, this setting is not available on the mobile app, but that's something we can consider adding in the future.
  2. If you have a direct message that has a mention badge on it, it should certainly be prioritized in your list of Direct Messages. @CptLolliPants https://github.com/CptLolliPants you've mentioned that the specific DM with a mention did not surface in your list of 20 DMs. Is that correct? If so, this may be a bug.

@enahum https://github.com/enahum anything to add?

— Reply to this email directly, view it on GitHub https://github.com/mattermost/mattermost-mobile/issues/7072#issuecomment-1416083376, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIY35K2JP4MAOZWO4CSMMKTWVUUZXANCNFSM6AAAAAAUNYOWLI . You are receiving this because you were mentioned.Message ID: @.***>

amyblais commented 1 year ago

Opened a ticket: https://mattermost.atlassian.net/browse/MM-50311.