-
Notifications
You must be signed in to change notification settings - Fork 44
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(home) : Android en version 7.2.0 et iOS en version 7.3.0 de la page d'accueil + Corrections de divers bugs #372
base: main
Are you sure you want to change the base?
Conversation
This reverts commit 5bcae66.
…ariables non utilisées
Comment ? |
non ?? |
Ben la majorité des personnes se plaignent d'une page d'accueil instable depuis la version 7.3 |
précision : sur android |
Je sais ! Elle pas forcément instable, j'ai pu tester sur 2 téléphone bas de gamme, ça fonctionne ! C'est juste effectivement moyennement stable ! MAIS c'est juste la tick speed qui est trop haute sur android ! |
Donc faut changer la tick speed ou faire un revert et reprendre après ? @godetremy |
On fix ! Je préfère que vous corriger plutôt que retirer ! |
ça dépend, elle était terminé au moment du merge ? autrement vaut mieux que tu la reprennes de ton côté @godetremy |
je vais voir ce que je peux faire, mais je garantis rien |
Sinon, il est possible de garder les composants avec |
OK je me le note pour quand j'ai le temps 👍 |
Comme ça ? |
t'as vraiment foutu un screen |
En tout cas, je rencontre pas ce problème sur l'ancienne page d'accueil avec Android |
Eh mais j'avais téléchargé la vidéo ! Bon je remets 391265969-5403b573-304b-4b8e-acde-fe4eb0da4bed.mp4 |
ok ça va dans ce cas, j'ai publié une version bêta sur le play store basé sur cette pr, elle est en cours d'examen (j'ai eu trop de plaintes sur la page d'accueil qui lag) |
Super 👌 |
Bon, comme ça n'a pas été merged, j'en ai profité pour corriger d'autres bugs (devoirs non listés) et intégrer une nouveauté (tout est en lien avec l'issue #391 ). Remplacement de
|
la vidéo est invalide @Kgeek33 |
|
invalide carrément ? ok je réessaye sur mon pc : 391548584-dabfbac0-a419-433e-b996-a2b28898e918.mp4 |
@Yaya-Cout comme t'arrives à voir la page d'accueil, penses-tu qu'il faut remplacer la page d'accueil montrée sur la vidéo ? |
c bon pour toi @LeGeek01 ? |
391548584-dabfbac0-a419-433e-b996-a2b28898e918.h264.mp4 |
pas mieux |
la c'est bon merci @Yaya-Cout |
Du coup, penses tu @LeGeek01 qu'il faut conserver cette nouvelle page d'accueil ? |
Bah au vu des bugs que je vois, visiblement elle semble assez potable sur ios même si y'a le bug du carré que ora a envoyé, sur android c'est affreux, je suis d'avis de ne pas la garder en production pour le moment jusqu'à ce que les bugs soit réglés |
Ou du moins de laisser comme dans cette pr, la nouvelle page sur ios et garder l'ancienne pour Android |
Oui je suis d'accord |
A la demande, je mets cette pr en draft pour faire un fix de la nouvelle page d'accueil plutôt que de faire un revert |
🚀 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].
TODO
(aka des annotations pour du code manquant) dans vos modificationsChangelogs proposés
Capture(s) d'écran
Issues en lien
Informations supplémentaires
Provenant de #325
Fait en partie par @Gabriel29306