Repository navigation
Conversation
Assisted-by: Codex:GPT-6 Signed-off-by: Li Wei <caeser222000@sohu.com>
Connect settings to LogsEmailSender and extend FileExportUtils with an optional Downloads subdirectory and reliable export results. Remove the separate exporter and controller while retaining background work, permission handling and failure feedback. Assisted-by: Codex:GPT-6 Signed-off-by: Li Wei <caeser222000@sohu.com>
Remove the debug logging switch, preferences, filtering and related tests. Preserve upstream logging behavior and reuse the existing export title. Assisted-by: Codex:GPT-6 Signed-off-by: Li Wei <caeser222000@sohu.com>
Keep the settings shortcut and legacy storage permission handling. Reuse the original export implementation with an optional Downloads subdirectory, removing the extra export behavior and its tests. Assisted-by: Codex:GPT-6 Signed-off-by: Li Wei <caeser222000@sohu.com>
xO-ox-ai
force-pushed
the
feat/settings-export-logs
branch
from
November 10, 2026 07:06
ffc510c to
5787590
Compare
Show the export directory and formatted file size in a toast and expandable notification after copying succeeds. Report export errors through the existing background runner instead of announcing success when writing fails. Assisted-by: Codex:GPT-6 Signed-off-by: Li Wei <caeser222000@sohu.com>
xO-ox-ai
force-pushed
the
feat/settings-export-logs
branch
from
November 12, 2026 03:43
d73ac15 to
4af81f3
Compare
xO-ox-ai
marked this pull request as ready for review
November 12, 2026 03:44
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds an Export logs shortcut to Settings → More that loads the existing logs and calls
LogsEmailSender.Log exports are saved in
Download/Nextcloud/. On completion, a toast and expandable notification show the destination and formatted file size. A failed export shows its error reason instead of a success notification. File copying runs in the existing background task, and write errors propagate to its error callback.The existing
FileExportUtilstakes an optional subdirectory; other file exports keep their default Downloads location. The shortcut handles storage permission on Android 9 and earlier. Log levels and logging defaults are unchanged.Validation
🖼️ Screenshots
Not provided; the settings shortcut and result messages have not been tested on a device or emulator.
🏁 Checklist
🤖 AI
Implementation and validation were assisted by Codex (GPT-6). The contributor has signed off every commit; AI assistance is recorded in the commit trailers.