-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Work app store #2553
Open
fynngodau
wants to merge
61
commits into
microg:master
Choose a base branch
from
e-foundation:workaccount-store
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+3,168
−574
Open
Work app store #2553
Changes from 1 commit
Commits
Show all changes
61 commits
Select commit
Hold shift + click to select a range
34ccbe2
SplitInstallService Additions
DaVinci9196 5e6f01e
add permission
DaVinci9196 1962f33
ISplitInstallServiceCallback added
DaVinci9196 c9bfbc0
remove Phenotype / Experiment
DaVinci9196 796c3ab
Merge branch 'microg:master' into split_install_service
DaVinci9196 e8a0058
Optimizing the code
DaVinci9196 b8d2b20
SplitInstallService add call super.onBind
DaVinci9196 6cdd488
Fixed the issue that multiple versions of language packs could not be…
DaVinci9196 36651db
Formatting Code
DaVinci9196 ff837d9
Formatting Code
DaVinci9196 8a68044
Execute processing suggestions
DaVinci9196 5a9f1b3
cleanup
DaVinci9196 7df6f93
Location/Huawei: Fix permission notification cancelling
mar-v-in bc57d78
Added the automatic login function for Games (#2435)
DaVinci9196 a7a2584
Location: Make sure current interval and configured interval stay in …
mar-v-in c5b7383
Create Work Account service
fynngodau 686ab6d
Introduce WorkAccountAuthenticatorService
jonathanklee 96e6b0c
Work account POC
fynngodau 7f4f252
Work profile: use correct AIDL
fynngodau 762fa53
Call account creation when app is using API to create work account
fynngodau 999e1d3
Work account: Fix rough edges
fynngodau 3af3991
Work account: Add security checks
fynngodau 5e2c9d3
Work account: Redeem token when signing in to account
fynngodau 3128c06
Add license identifiers and remove unwanted changes
fynngodau 30f4858
Work account: Fix `NewApi` lint failure
fynngodau 482038a
Deduplicate work account service declaration
fynngodau 5a39c65
POC: Test activity in companion app to show client policy
fynngodau 0db6eed
Improve visuals of vending activity
fynngodau c621098
Fetch bulk details for apps retrieved from policy
fynngodau 1e17cc1
Load app details using getItems
fynngodau 8e5ca13
Cleanup: Remove nonfunctional bulkDetails request
fynngodau 20e04fe
UI: Add download button
fynngodau e99cbd3
Fetch app download URLs
fynngodau 80c9c52
Add app installation support
fynngodau 7ab6cf1
Add app uninstallation support
fynngodau c3398b9
Refactor SplitInstallManager: data types
fynngodau 3edeb83
Refactor SplitInstallManager: structural pt. 1
fynngodau 23d4459
`FLAG_MUTABLE` instead of `FLAG_IMMUTABLE`
fynngodau e434c8c
Refactor SplitInstallManager: structural pt. 2
fynngodau 73f1a3a
Refactor SplitInstallManager: structural pt. 3
fynngodau 3b8509d
Work vending: Show installation success without reload
fynngodau 68ef487
Use cache dir instead of files dir
fynngodau f257e83
Enable work app store only after adding a work account
fynngodau 1887759
Fix lint
fynngodau 85bd1c9
Fix split installs
fynngodau 2e1dfec
Organize vending protobuf files
fynngodau ec18cdb
Replace volley with ktor + okhttp
fynngodau 09acd1a
Install packages directly to improve performance
fynngodau 9ee71fb
Add progress bar to work app store
fynngodau 352260d
Reimplement caching in `HttpClient`
fynngodau 2109e46
Discard session after exception to clear up storage
fynngodau c237146
Notifications upon install progress
fynngodau f39a889
Deduplicate auth-related files
fynngodau c976df1
Clean up constants
fynngodau f14124c
Merge master branch into workaccount-store
fynngodau 655cc6f
Fix crash when no apps in enterprise policy
fynngodau 4f0665c
Fix build
fynngodau a6dc3b3
Purchase apps before downloading in work store
fynngodau b2cec33
Support additional strange behaviors
fynngodau 04afa56
Show info: wait before work app store can be used
fynngodau ed745dc
Apply review
fynngodau File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fynngodau
I don't know the code enough but does it check the microG option before purchase?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That wouldn't make sense to me, as this is neither in-app billing nor during an app license check.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@fynngodau
If the existing option doesn't apply, I think we should add a new one, since anything that causes contact with a remote server (and thus potentially reduces privacy) should be able to be disabled by the user.