exdevutem / mi-utem

Aplicación móvil Mi UTEM de la Universidad Tecnológica Metropolitana de Chile
14 stars 6 forks source link

Actualización a Flutter 3, librerías y corrección de deprecaciones #11

Closed mapacheverdugo closed 1 year ago

mapacheverdugo commented 1 year ago

Descripción

Tipo de cambio

Checklist:

RafaelMoralesV commented 1 year ago

Estoy teniendo problemas para compilarlo con una vm corriendo android 33 (Tiramisú?)

Performing Streamed Install

adb: failed to install /home/rafael/proyectos/mi-utem/build/app/outputs/flutter-apk/app.apk: Failure [INSTALL_PARSE_FAILED_MANIFEST_MALFORMED: Failed parse during installPackageLI: /data/app/vmdl1795351735.tmp/base.apk (at Binary XML file line #198): me.carda.awesome_notifications.notifications.broadcastReceivers.RefreshNotificationBootReceiver: Targeting S+ (version 31 and above) requires that an explicit value for android:exported be defined when intent filters are present]

RafaelMoralesV commented 1 year ago

Estuve leyendo un poco del error; la recomendación típica es colocar android:exported='true' en algún lugar dentro de las Activity del android manifest. El tema es que nosotros ya lo tenemos de esa manera.

Pude cachar según el error y revisando los manifest intermedios que la wea está crasheando por la librería de notificaciones que tenemos agregada llamada awesome_notifications

Caché que la conseguimos a partir de un repo en particular, y que este es un fork del proyecto original, que está un 94% listo. El fork original si está listo para android y iOS, pero dice que tiene conflictos con firebase_messaging, que es una de las dependencias que tenemos (En el repo hablan un poco más al respecto)

Original: https://github.com/rafaelsetragni/awesome_notifications Fork utilizado: https://github.com/aaqibismail/awesome_notifications

Así que tenemos un problema aquí; no podemos buildear versiones nuevas de android a menos de que arreglemos o eliminemos esa dependencia

Propongo que no se solucione en esta branch; limitar la versión de android a la más nueva que permita compilar (probablemente el sdk 31 compila? lo revisaré en un rato), y generar una nueva branch que lidie con esto lo más pronto posible. Si les da paja a uds yo no tengo problema en intentar arreglarlo, pero demás voy a necesitar ayuda en revisar que todo esté funcionando

Puede que se me esté pasando algo tho, si saben de una forma de compilar las versiones nuevas de android coméntenmelas porfa

@mapacheverdugo @PollitoMayo

mapacheverdugo commented 1 year ago

Ya estoy trabajando en el error, e igual creo que mejor lo agreguemos acá porque es parte de actualización de librerías. El problema es justo lo que comentas, falta un exported=true pero a la librería, y es justamente porque el fork utilizado no está actualizado.

Prefiero dejar esta rama con la app sin problemas de build y sacrificar que las notificaciones funcionen al 100% y después podemos hacer pruebas y correcciones con las notificaciones en una nueva PR, hago push en un ratito

mapacheverdugo commented 1 year ago

@RafaelMoralesV fixed, creo

RafaelMoralesV commented 1 year ago

Pulento, reviso en un rato