Skip to content

Feature/merge imports#73

Open
LandWarderer2772 wants to merge 2 commits into
develfrom
feature/merge-imports
Open

Feature/merge imports#73
LandWarderer2772 wants to merge 2 commits into
develfrom
feature/merge-imports

Conversation

@LandWarderer2772
Copy link
Copy Markdown
Member

This pull request introduces a new "merge" option for restoring backups, allowing users to choose whether to merge settings and reader grid preferences with existing data or overwrite them. Additionally, it adds support for generating reading statistics grouped by genre (tag), and improves the UI and data handling for these features.

Backup Restore: Merge Option

  • Added a "merge" checkbox to the restore dialog UI (RestoreDialogFragment), allowing users to choose between merging or overwriting settings and reader grid preferences during backup restoration. The merge state is passed through the view model and service to the repository, and used to determine whether to clear existing preferences before upserting new ones. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12]

Reading Statistics: Group by Genre

  • Implemented support for reading statistics grouped by genre (tag) in the repository and DAO, including a new query method and logic to aggregate and display stats by tag, with small genres grouped under "Other". The StatsRecord model and UI were updated to support tag-based stats. [1] [2] [3] [4]

Codebase Maintenance

  • Reorganized and cleaned up imports across several files for better readability and consistency. [1] [2] [3] [4] [5] [6] [7] [8]

UI Improvements

  • Updated the stats adapter to display either the manga title, tag name, or a default label for "Other" when showing grouped statistics.

Utility Additions

  • Introduced a Quadruple data class for handling four-value tuples in the restore dialog's state management.

@github-actions
Copy link
Copy Markdown

Debug APK built successfully!

Download artifact: debug-apk

Artifact will be available for 7 days

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants