Skip to content
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

feat: Import and export manager settings #2268

Merged
merged 2 commits into from
Nov 5, 2024

Conversation

TheAabedKhan
Copy link
Member

@TheAabedKhan TheAabedKhan commented Oct 22, 2024

Export Manager Settings

Note

Patch options and patched apps info are also included.

Let me know if those values shouldn't be included.

@TheAabedKhan TheAabedKhan linked an issue Oct 22, 2024 that may be closed by this pull request
3 tasks
@indrastorms
Copy link

Instead of multiple import/export options show a dialog to select what(settings, patch selection) user wants to export would be clean way to do import/export

@TheAabedKhan
Copy link
Member Author

Instead of multiple import/export options show a dialog to select what(settings, patch selection) user wants to export would be clean way to do import/export

Thank you for the suggestion. However, this is irrelevant to this PR. You can open a separate issue regarding it so that everyone can discuss it properly, and we can track the enhancement separately. This will also help keep the scope of this PR focused and ensure smoother reviews.

Comment on lines 201 to 206
"exportManagerSettingsLabel": "Export manager settings",
"exportManagerSettingsHint": "Export manager settings to a JSON file",
"exportedManagerSettings": "Manager settings exported",
"importManagerSettingsLabel": "Import manager settings",
"importManagerSettingsHint": "Import manager settings from a JSON file",
"importedManagerSettings": "Manager settings imported",
Copy link
Member

Choose a reason for hiding this comment

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

Just saying "Export settings" is shorter. Adjust labels too.

Copy link
Member

@oSumAtrIX oSumAtrIX left a comment

Choose a reason for hiding this comment

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

Also add these settings to the docs under docs/ in this PR.Apart from the naming suggestion lgtm.

@validcube validcube self-requested a review October 26, 2024 09:01
@oSumAtrIX oSumAtrIX merged commit a45d959 into ReVanced:dev Nov 5, 2024
github-actions bot pushed a commit that referenced this pull request Nov 5, 2024
# [1.23.0-dev.5](v1.23.0-dev.4...v1.23.0-dev.5) (2024-11-05)

### Features

* Import and export manager settings ([#2268](#2268)) ([a45d959](a45d959))
github-actions bot pushed a commit that referenced this pull request Nov 10, 2024
# [1.23.0](v1.22.0...v1.23.0) (2024-11-10)

### Bug Fixes

* Don't show toasts when export cancelled ([#2230](#2230)) ([bd79496](bd79496))
* Fix the connectivity check toast again ([#2216](#2216)) ([a7e2281](a7e2281))
* Get changelogs for alternative sources ([#1766](#1766)) ([c729842](c729842))
* missing parameter in translations ([1c6c5d5](1c6c5d5))
* Resolve EACCESS error in special cases ([#2135](#2135)) ([1f95767](1f95767))
* Restore apk renaming during compile ([abdd9dc](abdd9dc))

### Features

* Also show new patches in the removed patches dialog ([#2257](#2257)) ([8872165](8872165))
* Import and export manager settings ([#2268](#2268)) ([a45d959](a45d959))
* Show changelogs from the latest to the last used patches version ([#2219](#2219)) ([daba737](daba737))
* Use ReVanced API v4 ([7b7d91d](7b7d91d))

### Performance Improvements

* Don't recalculate universal patches or compatible packages if not necessary ([7e3afe0](7e3afe0))
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.

feat: Add options to export & import manager's settings
3 participants