-
-
Notifications
You must be signed in to change notification settings - Fork 88
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
feat: Papillon 97% hors connexion (V2) #484
Conversation
Sinon, je peux faire comme pour ma pr sur la détection des tablettes, je fais un hook et ça simplifie considérablement le code |
…ppression d'imports/variables inutiles
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Voilà, pour un code bien optimisé, j'ai crée un hook pour avoir alors un code bien clean !
Déso encore des conflits 😅 |
mon dernier commit rend la page Discussions hors connexion, mais comme il n'y a pas de cache pour les messages, la page ne fait rien du tout et la création d'un message est bloqué en disant de vérifier la connexion internet (comme si on met une actualité en non lu et comme si on cochait un devoir) Voir => https://github.com/user-attachments/assets/3b329a5f-9cfe-45ba-942a-f4c613df26ed |
@Bulgus @ecnivtwelve est-ce que je dois tutoyer l'utilisateur pour éviter d'être en conflit avec #460 ? |
Désolé du temps de réponse 😓 |
Tkt pas. OK je fais ça plus tard ;) |
Note Cette review est le résultat de la concertation de toute l'équipe Papillon. Test ExpoPas possible avec un simulateur je vais péter un cable... Résultat du codeCode superfluNous avons trouver du code superflu dans ta PR, comme les widget, le lint, et compagnie... Fonction répétitiveEssaye de simplifier tes copier coller en fonction |
ok merci de la review ! |
🚀 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
L'application est disponible à 100% hors connexion ! En fonction de la page, soit est affiché une icône
WifiOff
, soitReanimated.View
avec un titre au hasard et un texte disant que l'utilisateur est en mode hors connexionIssues en lien
Captures d'écran/Vidéos
Informations supplémentaires
Warning
Quand l'utilisateur est en mode hors connexion, et qu'il se reconnecte, il y aura un chargement car Papillon n'aura pas réussi à se connecter. J'ai cherché des moyens pour raffraichir des pages, sans succès
Il faut redémarrer l'app pour que ça fonctionne comme attendu
Copier-coller de #346