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

feat: Papillon 97% hors connexion (V2) #484

Closed
wants to merge 65 commits into from

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Dec 16, 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

L'application est disponible à 100% hors connexion ! En fonction de la page, soit est affiché une icône WifiOff, soit Reanimated.View avec un titre au hasard et un texte disant que l'utilisateur est en mode hors connexion

Issues en lien

Captures d'écran/Vidéos

Page Image/Vidéo
Home Home
Devoirs Devoirs
Cocher un devoir (même coche pour mettre une actualité en lu/non lu) https://github.com/user-attachments/assets/3b329a5f-9cfe-45ba-942a-f4c613df26ed
Emploi du temps Lessons
Notes Notes
Actualités Actualités
Cantine (pareil pour Messages) Cantine
Ajouter un compte https://github.com/user-attachments/assets/b5d9f830-3745-42a8-b9ad-7f4a7e583fb2

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

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 25, 2024

si c'est pas clean n'approuve pas

si c'est relativement OK, je considère pas ça si problématique tant qu'on comprend ce qu'il se passe et que c'est pas non plus illogique

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

Copy link
Contributor Author

@Kgeek33 Kgeek33 left a 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 !

@Kgeek33 Kgeek33 requested a review from ecnivtwelve December 25, 2024 22:40
ecnivtwelve
ecnivtwelve previously approved these changes Dec 26, 2024
ecnivtwelve
ecnivtwelve previously approved these changes Dec 29, 2024
@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 29, 2024

Déso encore des conflits 😅

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 30, 2024

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

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 30, 2024

@Bulgus @ecnivtwelve est-ce que je dois tutoyer l'utilisateur pour éviter d'être en conflit avec #460 ?

@Bulgus
Copy link
Contributor

Bulgus commented Jan 6, 2025

@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 😓
Oui, puisque c'est comme ça que c'est fait sur Figma ;)
D'ailleurs je ne sais pas ou en est la PR pour les templates de PR, mais je n'avais pas réussi à ajouter la case "Les messages tutoient l'utilisateur"

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 6, 2025

@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 😓
Oui, puisque c'est comme ça que c'est fait sur Figma ;)
D'ailleurs je ne sais pas ou en est la PR pour les templates de PR, mais je n'avais pas réussi à ajouter la case "Les messages tutoient l'utilisateur"

Tkt pas. OK je fais ça plus tard ;)
En attente de merge mais j'intègre ça de suite

@godetremy
Copy link
Contributor

Note

Cette review est le résultat de la concertation de toute l'équipe Papillon.

Test Expo

Pas possible avec un simulateur je vais péter un cable...

Résultat du code

Code superflu

Nous avons trouver du code superflu dans ta PR, comme les widget, le lint, et compagnie...

Fonction répétitive

Essaye de simplifier tes copier coller en fonction

@godetremy godetremy closed this Jan 11, 2025
@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Jan 11, 2025

ok merci de la review !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Enhancement]: Mode hors connexion (en cas de bug ou de manque de connexion)
7 participants