Skip to content

Conversation

@BijinDev
Copy link
Contributor

@BijinDev BijinDev commented Apr 8, 2025

Since there is not a separate db for searching when offline storage is available, having the "Search mailbox" option does not make sense, since search is always enabled.
The option remains for web version.

Close: #8834

Test notes

  • "Search mailbox" setting is hidden from settings/email view on desktop and mobile and search is always enabled
  • "Search mailbox" setting is visible on web and it works as expected

Base automatically changed from sqlite-search to search-integration April 9, 2025 08:16
@hrb-hub hrb-hub force-pushed the 8834-SearchIndexing-option-only-in-browser branch from ffe9121 to aade48b Compare April 9, 2025 08:27
@hrb-hub hrb-hub linked an issue Apr 9, 2025 that may be closed by this pull request
4 tasks
Copy link
Contributor

@wrdhub wrdhub left a comment

Choose a reason for hiding this comment

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

Just need some test notes on the issue

@hrb-hub hrb-hub force-pushed the 8834-SearchIndexing-option-only-in-browser branch from aade48b to 40008a7 Compare April 11, 2025 08:40
Since there is not a separate db for searching when offline storage is
available, having the "Search mailbox" option does not make sense, since
search is always enabled.
The option remains for web version.

Close: #8834

Co-authored-by: hrb-hub <hrb-hub@users.noreply.github.com>
@hrb-hub hrb-hub force-pushed the 8834-SearchIndexing-option-only-in-browser branch from 40008a7 to dc72b87 Compare April 14, 2025 12:34
@hrb-hub hrb-hub merged commit dc72b87 into search-integration Apr 14, 2025
6 checks passed
@hrb-hub hrb-hub deleted the 8834-SearchIndexing-option-only-in-browser branch April 14, 2025 13:13
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.

Update disabling search option in Apps

4 participants