Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Skip to content

Removed uncessary delays of install process

Hasib Prince requested to merge 5713-technical_debt into main

Description

There were some unnecessary delays to avoid race conditions. Those delays are removed.

Handled app install error: INSTALL_PARSE_FAILED_NOT_APK

Issue(s)

backlog/-/issues/5713

Test environment

Gradle build against 5713-technical_debt branch, install the apk into the device

Test procedures

Technical details

After removing delays, App was facing an exception in Room. It happened because of an unnecessary status update, that interrupted the getDownloadListLiveData() call in MainActivity. The unnecessary status update has been removed.

App installs faced an apk parse error because of a CleanApk API issue, it was returning an unexpected apk file. When this error happens, the bound service with PackageInstaller receives an empty package name. Package name is required to mark the app has faced an error. That's why package name is added from PkgManagerModule to make packagname always available in the bounded service.

Screenshots

Check list

  • Self review
  • Test procedure explained
  • Tested on fresh install
  • Tested applied as an update
  • License
  • Internal documentation
  • User documentation
Edited by Hasib Prince

Merge request reports

Loading