Skip to content

Conversation

@Jozys
Copy link
Collaborator

@Jozys Jozys commented May 5, 2025

MoveTopia Update Changelog

Removed Functionalities

  • Temporary Removed shown source app icon from activities due to play store restrictions

Entfernte Funktionen

  • Quell-App-Icon von Aktivitäten temporär entfernt aufgrund von Play Store Restriktionen

Jozys added 3 commits May 5, 2025 09:02
- This is due to QUERY_ALL_PACKAGES not being allowed to be used in production for our use case
# 🚀 Pull Request

## Brief Description
This pull request removes, with sadness, the shown icons in the
activities list due to the QUERY_ALL_PACKAGES not being allowed to be
used in production for our use case.

## GitHub Copilot Text
This pull request removes the usage of the `installed_apps` package and
its related functionality due to restrictions on the
`QUERY_ALL_PACKAGES` permission, which is not allowed for release on the
Play Store. The most important changes include disabling the app icon
fetching logic and removing the `installed_apps` dependency.

### Removal of `installed_apps` functionality:

*
[`lib/utils/system_utils.dart`](diffhunk://#diff-fa53e970602244b2a9aef530e233a7a1e27b1d588854f4dcd1a4675529455f26L3-R16):
Disabled the `getInstalledAppIcon` method by commenting out the logic
that fetches app icons using the `installed_apps` package. Added a
comment explaining that the method is not usable in production due to
Play Store restrictions on the `QUERY_ALL_PACKAGES` permission.

### Dependency cleanup:

*
[`pubspec.yaml`](diffhunk://#diff-8b7e9df87668ffa6a04b32e1769a33434999e54ae081c52e5d943c541d4c0d25L28):
Removed the `installed_apps` dependency (`^1.5.2`) since it is no longer
used.
@Jozys Jozys requested a review from ScreepCode May 5, 2025 07:22
@Jozys Jozys self-assigned this May 5, 2025
@Jozys Jozys merged commit 14de038 into main May 5, 2025
3 checks passed
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.

3 participants