Skip to content

fix: Only refresh mailboxes with displayed notifications #1634

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

Merged
merged 1 commit into from
Jan 8, 2025

Conversation

PhilippeWeidmann
Copy link
Member

No description provided.

Copy link

github-actions bot commented Jan 8, 2025

Found 0 unused code occurences

Expand
* No unused code detected.

Copy link
Contributor

@adrien-coye adrien-coye left a comment

Choose a reason for hiding this comment

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

Have a look at the notes and let me know what you think.
I'll test the code after you get back to me.

@PhilippeWeidmann PhilippeWeidmann force-pushed the fix-unread-notification-fetching branch from 04eb7d6 to c5fb128 Compare January 8, 2025 13:09
@PhilippeWeidmann PhilippeWeidmann force-pushed the fix-unread-notification-fetching branch from c5fb128 to b634915 Compare January 8, 2025 13:17
Copy link

sonarqubecloud bot commented Jan 8, 2025

Copy link
Contributor

@adrien-coye adrien-coye left a comment

Choose a reason for hiding this comment

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

All good for me. I did test it for a while.
It looks like it does not work in Lckdwn mode, but that's beyond our control.

@PhilippeWeidmann PhilippeWeidmann merged commit aa277c0 into master Jan 8, 2025
10 checks passed
@PhilippeWeidmann PhilippeWeidmann deleted the fix-unread-notification-fetching branch January 8, 2025 14:11
@adrien-coye adrien-coye restored the fix-unread-notification-fetching branch January 8, 2025 14:36
@adrien-coye adrien-coye deleted the fix-unread-notification-fetching branch January 8, 2025 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants