Multiple improvements to Apps

See merge request e/apps/apps!60

Currently translated at 99.1% (123 of 124 strings)

Translation: /e//Apps
Translate-URL: https://i18n.e.foundation/projects/e/apps/eu/

Signed-off-by: Aayush Gupta <theimpulson@e.email>

res/drawable/ directory should only contains xml based drawables

This change resolves multiple lint warnings
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Current launcher icon is actually a .jpg file renamed as .png. This is evident because
Android Studio fails the builds with an error and lint warns regarding same as well.

Use an online converter to convert the image and drop the gradle flag which is no
longer required
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Resolves lint warning
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Resolves multiple lint warnings
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Signed-off-by: Aayush Gupta <theimpulson@e.email>

This will allow us to install debug builds without removing release builds
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Current method has been deprecated

References:
- [1]: https://developer.android.com/reference/android/os/Environment#getExternalStorageDirectory()

Signed-off-by: Aayush Gupta <theimpulson@e.email>

Signed-off-by: Aayush Gupta <theimpulson@e.email>

Observed several warnings in compilation otherwise
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Warnings observed during compilation

w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/application/model/Application.kt: (98, 29): Variable 'error' initializer is redundant
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Warnings observed in Android Studio

w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/application/model/StateManager.kt: (46, 86): Elvis operator (?:) always returns the left operand of non-nullable type String
w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/application/model/StateManager.kt: (79, 82): Elvis operator (?:) always returns the left operand of non-nullable type String
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Current method has been deprecated

w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/utils/Common.kt: (47, 53): 'getter for activeNetworkInfo: NetworkInfo!' is deprecated. Deprecated in Java
w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/utils/Common.kt: (48, 63): 'getter for isConnected: Boolean' is deprecated. Deprecated in Java
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Resolves multiple compile-time warnings
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Current method has been deprecated

References:
- [1]: https://developer.android.com/reference/android/graphics/drawable/Drawable#setColorFilter(int,%20android.graphics.PorterDuff.Mode)0

Signed-off-by: Aayush Gupta <theimpulson@e.email>

w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/XAPK/InstallSplitApksActivity.kt: (83, 20): Unnecessary non-null assertion (!!) on a non-null receiver of type PackageInstaller.Session
Signed-off-by: Aayush Gupta <theimpulson@e.email>

We have migrated to require API making these checks useless
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Above if-else conditional already handles all cases making this boolean is unreachable

w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/application/model/Application.kt: (202, 9): Unreachable code
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Current method has been deprecated and alternative is limited to API 28

References:
- [1]: https://developer.android.com/reference/android/content/pm/PackageInfo#versionCode
- [2]: https://developer.android.com/reference/androidx/core/content/pm/PackageInfoCompat#getLongVersionCode(android.content.pm.PackageInfo)

Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Compile-time warnings observed

w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/XAPK/XApkInstallUtils.kt: (144, 22): Unsafe use of a nullable receiver of type File?
w: /builds/e/apps/apps/app/src/main/java/foundation/e/apps/XAPK/XApkInstallUtils.kt: (145, 21): Unsafe use of a nullable receiver of type File?
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Currently used method has been deprecated

References:
- [1]: https://developer.android.com/reference/android/net/NetworkInfo

Signed-off-by: Aayush Gupta <theimpulson@e.email>

Current method has been deprecated

References:
- [1]: https://developer.android.com/reference/android/content/res/Resources#getDrawable(int)

Signed-off-by: Aayush Gupta <theimpulson@e.email>

The current one has been deprecated

References:
- [1]: https://developer.android.com/reference/android/content/res/Resources#getColor(int)

Signed-off-by: Aayush Gupta <theimpulson@e.email>

getInstance() method has been deprecated

References:
- [1]: https://developer.android.com/reference/androidx/work/WorkManager.html#getInstance()

Signed-off-by: Aayush Gupta <theimpulson@e.email>

The current one has been deprecated

References:
- [1]: https://developer.android.com/reference/kotlin/android/preference/PreferenceManager

Signed-off-by: Aayush Gupta <theimpulson@e.email>

There is nothing of importance to backup in Apps and keeping this
setting on without providing proper backup rules results in lint warning
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Improves the app's performance
- Warned by android's lint checks
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Signed-off-by: Aayush Gupta <theimpulson@e.email>

minSdkVersion is high enough for us to drop these checks
Signed-off-by: Aayush Gupta <theimpulson@e.email>

AndroidX added new "require____()" versions of common "get___()" APIs, such as getContext/getActivity/getArguments/etc.
Rather than wrap these in something like requireNotNull(), using these APIs will allow the underlying component to try
to tell you _why_ it was null, and thus yield a better error message.
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Resolves mutiple lint errors

References:
- [1]: https://stackoverflow.com/questions/59521691/use-viewlifecycleowner-as-the-lifecycleowner

Signed-off-by: Aayush Gupta <theimpulson@e.email>

Signed-off-by: Aayush Gupta <theimpulson@e.email>

Resolves multiple lint errors
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Resolves multiple lint warnings
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- gradle gives compile-time errors otherwise
Signed-off-by: Aayush Gupta <theimpulson@e.email>

- Bump buildToolsVersion, compileSdkVersion, and targetSdkVersion to 29 (We are currently shipping on Android 10)
- Bump minSdkVersion to 25 (We are only backwards compatiable till Android 7.1)
Signed-off-by: Aayush Gupta <theimpulson@e.email>

These contains important reports like lint which is useful for
debugging issues
Signed-off-by: Aayush Gupta <theimpulson@e.email>

Signed-off-by: Aayush Gupta <theimpulson@e.email>