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

read-only mailing lists should not appear in share/forward list #4616

Closed
r10s opened this issue Aug 10, 2023 · 0 comments · Fixed by #4617
Closed

read-only mailing lists should not appear in share/forward list #4616

r10s opened this issue Aug 10, 2023 · 0 comments · Fixed by #4617
Assignees
Labels
bug Something is not working

Comments

@r10s
Copy link
Member

r10s commented Aug 10, 2023

in general, read-only chats do not appear at dc_get_chatlist(ctx, DC_GCL_FOR_FORWARDING ...), however, read-only-mailing lists are listed here, maybe as we do not have a "hard" internal flag for that, iirc, this is hard to detect and may change.

this issues was extracted from #4612

@r10s r10s added the bug Something is not working label Aug 10, 2023
@iequidoo iequidoo self-assigned this Aug 11, 2023
iequidoo added a commit that referenced this issue Aug 11, 2023
…ere we can send (#4616)

I.e. exclude from the list the following chats as well:
- Read-only mailing lists.
- Chats we're not a member of.
- Chats with broken protection.
iequidoo added a commit that referenced this issue Aug 12, 2023
…ere we can send (#4616)

I.e. exclude from the list the following chats as well:
- Read-only mailing lists.
- Chats we're not a member of.

But as for ProtectionBroken chats, we return them, as that may happen to a verified chat at any
time. It may be confusing if a chat that is normally in the list disappears suddenly. The UI need to
deal with that case anyway.
iequidoo added a commit that referenced this issue Sep 1, 2023
…ere we can send (#4616)

I.e. exclude from the list the following chats as well:
- Read-only mailing lists.
- Chats we're not a member of.

But as for ProtectionBroken chats, we return them, as that may happen to a verified chat at any
time. It may be confusing if a chat that is normally in the list disappears suddenly. The UI need to
deal with that case anyway.
iequidoo added a commit that referenced this issue Sep 1, 2023
…ere we can send (#4616)

I.e. exclude from the list the following chats as well:
- Read-only mailing lists.
- Chats we're not a member of.

But as for ProtectionBroken chats, we return them, as that may happen to a verified chat at any
time. It may be confusing if a chat that is normally in the list disappears suddenly. The UI need to
deal with that case anyway.

(cherry picked from commit 83ef25e)
iequidoo added a commit that referenced this issue Sep 1, 2023
…ere we can send (#4616)

I.e. exclude from the list the following chats as well:
- Read-only mailing lists.
- Chats we're not a member of.

But as for ProtectionBroken chats, we return them, as that may happen to a verified chat at any
time. It may be confusing if a chat that is normally in the list disappears suddenly. The UI need to
deal with that case anyway.

(cherry picked from commit 83ef25e)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
None yet
2 participants