Skip to content
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

fix(Android) : futur conflit #314

Closed
wants to merge 3 commits into from
Closed

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Oct 27, 2024

🚀 Nouvelle Pull Request

Proposez vos modifications pour améliorer Papillon

Informations importantes

Merci de vous référer à la documentation sur la contribution si vous avez des questions à propos des pull requests (https://gitbook.getpapillon.xyz/organisation/outils-internes/github)

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Vous respectez les conventions de codage et de nommage du projet
  • Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
  • Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

Comme à chaque maj de Papillon, les mises à jour de l'app sur Android pose problème, car la versionCode ne correspond pas à celle du Play Store
En ce moment, @LeGeek01 a mis une version code de "7211", alors je la passe à "7300"

Informations supplémentaires

Va falloir rebuider l'apk + l'aab pour éviter les problèmes sur le Play Store

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 27, 2024

@ecnivtwelve en rapport avec ma pr, faut que tu changes ceci pour que Github Actions fonctionne pour le Play Store :
Screenshot_2024-10-28-00-11-27-165_com.github.android.jpg

@LeGeek01
Copy link
Contributor

j'ai vu que la mise à jour 7.3.0 a été publiée sur le play store, je vais aller voir si c'est la bonne version

@LeGeek01
Copy link
Contributor

celui qui a mis 7.2.1 sur la version 7.3.0 est un big brain mdr
image

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

j'ai vu que la mise à jour 7.3.0 a été publiée sur le play store, je vais aller voir si c'est la bonne version

Ah bon ?
Je l'ai pas pu alors, les changelogs n'ont pas été changés

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

celui qui a mis 7.2.1 sur la version 7.3.0 est un big brain mdr
image

Ohlala 😂😭

@LeGeek01
Copy link
Contributor

j'ai vu que la mise à jour 7.3.0 a été publiée sur le play store, je vais aller voir si c'est la bonne version

Ah bon ? Je l'ai pas pu alors, les changelogs n'ont pas été changés

l'update est proposée, mais c'est la version 7.0.0 qui est installée et visiblement celui qui a upload la version a repris mes anciens changelogs, je suis en train de rebuild l'appli

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Oh ça recommence ? C'est bizarre ça quand même

@LeGeek01
Copy link
Contributor

bah quand je build l'appli j'ai bien les changements, donc bon mdr

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

C'est encore plus chelou

@LeGeek01
Copy link
Contributor

mon build de la 7.3.0 est parti en examen chez google

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Super
Faudrait faire ou changer le Github actions pour éviter ces problèmes là, c'est chiant à chaque fois

@LeGeek01
Copy link
Contributor

bah y'a pas un soucis de perm qui empêche actions d'upload les builds ?

@LeGeek01
Copy link
Contributor

moi je pense que le soucis est humain, je sais pas qui upload des versions erronées sur le play store

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

bah y'a pas un soucis de perm qui empêche actions d'upload les builds ?

Ben depuis septembre, la version code reste bloqué à 1, du coup, les apk sont foireuses si t'as une version code >
Genre ajouter dans le Github actions d'augmenter la version code de 100 à chaque nouvelle version

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

moi je pense que le soucis est humain, je sais pas qui upload des versions erronées sur le play store

Oui c'est aussi un problème, une mauvaise compréhension du play store 🤷

@LeGeek01
Copy link
Contributor

bah y'a pas un soucis de perm qui empêche actions d'upload les builds ?

Ben depuis septembre, la version code reste bloqué à 1, du coup, les apk sont foireuses si t'as une version code > Genre ajouter dans le Github actions d'augmenter la version code de 100 à chaque nouvelle version

je parle surtout du play store, parce que le version code est bien mis à jour, mais le bundle correspond à la 7.0.0 pour une raison que j'ignore

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Oui ça aussi c'est bizarre
Tjrs pas reçu la maj, ils sont lent un peu Google

@LeGeek01
Copy link
Contributor

La màj est dispo sur le play store

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Merci !

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Non il y a un problème @LeGeek01
C'est la version 7.2.1 qui a été publié

@LeGeek01
Copy link
Contributor

Non il y a un problème @LeGeek01
C'est la version 7.2.1 qui a été publié

Ben ? Je viens de faire la màj je suis bien en 7.3.0 moi

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Screenshot_2024-10-28-13-37-38-049_xyz.getpapillon.app.jpg

Un exemple

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Il y a juste les paramètres de Papillon qui ont changé avec la dernière version
Sinon, rien d'autre. Tout est resté en 7.2.1

@LeGeek01
Copy link
Contributor

Ah oui, pourtant je suis à jour sur la branche main
Je vérifie en rentrant chez moi

@ecnivtwelve
Copy link
Contributor

c'est moi qui a push l'update, c'est bien la version 7.3.0 normalement, j'ai juste zappé de le changer dans build.gradle

@LeGeek01
Copy link
Contributor

LeGeek01 commented Oct 28, 2024

c'est moi qui a push l'update, c'est bien la version 7.3.0 normalement, j'ai juste zappé de le changer dans build.gradle

Sur la 7.3.0 que j'ai build la page paramètres fonctionne tandis que sur la version que tu as upload elle est grise...
J'avais aussi l'invite de configuration de la cantine que je n'ai pas sur la 7.3.0

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Du coup, qu'est-ce qui n'est pas bon sur le Play Store en ce moment ?

@LeGeek01
Copy link
Contributor

Du coup, qu'est-ce qui n'est pas bon sur le Play Store en ce moment ?

Bah ed qui fonctionne pas visiblement

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 28, 2024

Du coup, qu'est-ce qui n'est pas bon sur le Play Store en ce moment ?

Bah ed qui fonctionne pas visiblement

Du coup, il faut refaire l'apk ?

@LeGeek01
Copy link
Contributor

Je vais rebuild l'apk quand je pourrais

@LeGeek01
Copy link
Contributor

ok j'ai compris, j'ai build sur ma branche main qui n'est pas à jour ptdr je viens de relancer le build sur la bonne branche

@LeGeek01
Copy link
Contributor

bonjour, la bonne version est dispo sur le play store

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Oct 29, 2024

bonjour, la bonne version est dispo sur le play store

Salut, c'est bien la bonne version, merci !

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 3, 2024

Intégration de ce fix sur une autre PR fixant tous les bugs (bientôt dispo, je prends le relai de @Gabriel29306 sur la PR #294)

Kgeek33@3340e5a

@Kgeek33 Kgeek33 closed this Nov 3, 2024
@Kgeek33 Kgeek33 deleted the fix/android branch November 3, 2024 14:49
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.

4 participants