Skip to content

Conversation

@Simon-Laux
Copy link
Member

@Simon-Laux Simon-Laux commented Jan 11, 2026

made for solving
deltachat/deltachat-desktop#5891 (comment)

will also be more efficient, because desktop currently loads all fresh messages to find out which chats to mark as noticed. https://github.com/deltachat/deltachat-desktop/blob/76d32bfc9364d53e1ffea40fcf4a60ad9a8482ee/packages/frontend/src/components/AccountListSidebar/AccountItem.tsx#L334

progress

@Simon-Laux Simon-Laux requested a review from WofWca January 11, 2026 06:57
Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Jan 11, 2026
Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Jan 11, 2026
Copy link
Collaborator

@WofWca WofWca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks sensible overall, didn't test.

Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Jan 11, 2026
@Simon-Laux Simon-Laux marked this pull request as ready for review January 12, 2026 12:50
@Simon-Laux Simon-Laux requested review from Hocuri and iequidoo January 12, 2026 12:51
Simon-Laux and others added 2 commits January 12, 2026 12:51
Co-authored-by: WofWca <wofwca@protonmail.com>
@Simon-Laux Simon-Laux added the bug Something is not working label Jan 15, 2026
Copy link
Collaborator

@iequidoo iequidoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, however, i didn't dive into what is wrong with the implicit start_chat_ephemeral_timers() calls

@Simon-Laux Simon-Laux requested a review from r10s January 15, 2026 22:36
@Simon-Laux Simon-Laux merged commit 4ccd3cb into main Jan 20, 2026
30 checks passed
@Simon-Laux Simon-Laux deleted the simon/mark-noticed-all branch January 20, 2026 08:53
Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Jan 23, 2026
Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Jan 26, 2026
Simon-Laux added a commit to deltachat/deltachat-desktop that referenced this pull request Jan 28, 2026
…5923)

* fix: muted chats were not marked read when marking account as read

closes #5891

depends on:
- core with chatmail/core#7709
- should be merged after #5922

* rm unused import

* fix: avoid undefined error in Object.keys

---------

Co-authored-by: Nico de Haen <ndh@deltachat.de>
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

Development

Successfully merging this pull request may close these issues.

4 participants