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: Add a toggle for alternative sources #1686

Merged
merged 6 commits into from Feb 19, 2024
Merged

feat: Add a toggle for alternative sources #1686

merged 6 commits into from Feb 19, 2024

Conversation

TheAabedKhan
Copy link
Member

@TheAabedKhan TheAabedKhan commented Feb 12, 2024

Some Screenshots

Before After (Disabled) After (Enabled)
image image image

Test

  • When using the alternative source, the patches card in home updates accordingly and the patches do too
  • Casing has no effect for alternative sources
  • Resetting works
  • Migration works

TheAabedKhan and others added 3 commits February 12, 2024 23:59
# Conflicts:
#	assets/i18n/en_US.json
#	lib/services/manager_api.dart
#	lib/ui/views/settings/settingsFragment/settings_manage_sources.dart
#	lib/ui/widgets/settingsView/settings_advanced_section.dart
@oSumAtrIX
Copy link
Member

@Aunali321 the checkboxes above are unticked. Have you tested them?

@Aunali321
Copy link
Member

@oSumAtrIX @TheAabedKhan at the time of review, you asked me to do code review after i raised an issue with UI. Today i go ahead with full test and found only single issue. After turning on alternative sources toggle, changing sources and turning it off, it shows no applications found. It should show default patches.

@TheAabedKhan
Copy link
Member Author

After turning on alternative sources toggle, changing sources and turning it off, it shows no applications found. It should show default patches.

Have you tried restarting the app? Can you attach a screen recording of the test?

@Aunali321
Copy link
Member

Screen_Recording_20240219_102840_ReVanced.Manager.Debug.mp4

@Aunali321
Copy link
Member

Fixed.

@TheAabedKhan TheAabedKhan merged commit f89c742 into dev Feb 19, 2024
1 check passed
github-actions bot pushed a commit that referenced this pull request Feb 19, 2024
## Features
* Add a toggle for alternative sources (#1686) ([aAbed](f89c742))
@TheAabedKhan TheAabedKhan deleted the fix/sources branch February 19, 2024 15:31
@Daredevil74

This comment was marked as off-topic.

github-actions bot pushed a commit that referenced this pull request Mar 5, 2024
# [1.19.0](v1.18.0...v1.19.0) (2024-03-05)

### Bug Fixes

* added a trailing comma ([975180b](975180b))
* adjust padding ([3559477](3559477))
* Allow mounting without Magisk ([3f96608](3f96608))
* Bump dependencies to support BCS keystore ([6ec6546](6ec6546))
* Do not delete files from post-fs-data.d ([70a1086](70a1086))
* Fix patched APKs exports after installation ([1200360](1200360))
* fix redundant buttons on dialog ([079c0de](079c0de))
* Incorrect strings and logics ([#1619](#1619)) ([4f22e88](4f22e88))
* **Keystore Password:** textfield title display ([8e52abd](8e52abd))
* Mount script causes build to fail ([#1613](#1613)) ([f3c78c2](f3c78c2))
* **Patch Option:** Set text colour on dropdown menu ([acb1e24](acb1e24))
* **PopScope:** User able to exit patch screen when the installer is still running ([#1663](#1663)) ([eb6d3cd](eb6d3cd))
* **Release CI:** truncate the "v" from version ([8595099](8595099))
* Show version label correctly ([c72d10a](c72d10a))
* Specify that dark theme is dark ([#1699](#1699)) ([d4b15ae](d4b15ae))
* Stop patch when signing fails ([#1553](#1553)) ([5b2c551](5b2c551))
* **Update Confirmation Sheet:** Add top padding ([9aeb156](9aeb156))
* Use correct title size for bottom sheet ([#1687](#1687)) ([3436523](3436523))
* Use correct version code & name ([#1647](#1647)) ([d933997](d933997))
* use lowercase repo names ([#1626](#1626)) ([edd8602](edd8602))

### Features

* Add a toggle for alternative sources ([#1686](#1686)) ([f89c742](f89c742))
* Add API migration code ([#1615](#1615)) ([28ae276](28ae276))
* add haptic feedback ([#1459](#1459)) ([7911459](7911459))
* Allow changing languages ([#1488](#1488)) ([f82c439](f82c439))
* Display current app language at the top of the list ([aa0575a](aa0575a))
* Hide the Install button during installation ([#1633](#1633)) ([3e696d6](3e696d6))
* Improve app selector and patcher UI ([#1616](#1616)) ([efb2d5e](efb2d5e))
* Improve consistency on language selector ([b2119ce](b2119ce))
* Improve language distinguishness and resolve language-specific issues ([#1706](#1706)) ([6d866d4](6d866d4))
* Improve Split APK warning readability ([#1625](#1625)) ([6fd740f](6fd740f))
* Show a dialog when an update is available ([#1654](#1654)) ([c7d975e](c7d975e))
* Simplify settings strings ([#1618](#1618)) ([0d45fe4](0d45fe4))
* Use more appropriate progress values ([2d7026a](2d7026a))
* use native TextField ([9ed43ef](9ed43ef))

### Performance Improvements

* Load patched apps as soon as possible ([c94eb7a](c94eb7a))

### Reverts

* WillPopScope migration ([3b0fed5](3b0fed5))
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.

None yet

4 participants