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 100% hors connexion #346

Open
wants to merge 44 commits into
base: main
Choose a base branch
from

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Nov 10, 2024

Participation de @imyanice dans cette PR

🚀 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

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

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

@camarm-dev
Copy link
Member

Essaye de faire passer le linter / typecheck ma PR a été merge... Je suis en train de tester mais ça charge dans le vide expo go là

@camarm-dev
Copy link
Member

Ah nan ma PR est pas merge mb

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 10, 2024

non ta pr n'est pas passé, mais t'as des erreurs de typages dans ta pr 🤣

@camarm-dev
Copy link
Member

non ta pr n'est pas passé, mais t'as des erreurs de typages dans ta pr 🤣

Nan elle est clean ma PR

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 10, 2024

il y a 50 min, vince a fait un commit sur ta pr qui fait que t'as des erreurs de typages désormais

@camarm-dev
Copy link
Member

oh nan j'vais pas vu pourquoi

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 10, 2024

Il a corrigé des bugs mais j'ai pas vu en détail pourquoi des erreurs de typage

@imyanice
Copy link
Contributor

Perso je trouve qu'il y'a beaucoup trop d'indicateur hors connexion et c'est moche, de plus entre Messages et Notes c'est pas centré pareil.

image
imageimageimage

@imyanice
Copy link
Contributor

Je pense que l'on peut enlever l'indicateur rouge et juste garder le header "Flûte"/"Catastrophe" sur chaque page, je trouve ça moins agressif...

@imyanice
Copy link
Contributor

De plus, les chats ne sont pas disponibles hors connexion

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 10, 2024

ok je corrige ça 👍, je laisse le wifi en rouge sur les devoirs ou pas ?

Ah oui, j'ai oublié de changer le texte pour le chat 😅

@imyanice
Copy link
Contributor

ok je corrige ça 👍, je laisse le wifi en rouge sur les devoirs ou pas ?

Je ne pense pas que ça soit utile! Rajoute juste le header Flûte! ça suffit imo

Sinon very cool la pr 😄

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 10, 2024

ok je fais ça
haha merci 😃

@ecnivtwelve
Copy link
Contributor

y'a pas mal de pr assez sympa en terme de fonctionnalités mais les review sont pauvres, c'est dommage parce que ça traine et on peut pas avancer

Ils sont vraiment incompétents les gens de Papillon dis donc

@LeGeek01
Copy link
Contributor

Sur certains points, oui, très clairement, notamment le manque de communication, point que j'ai souligné et rabaché et qui m'a coûté ma place au sein de l'équipe, et il n'y a pas d'amélioration.

Donc oui, à un moment, ça devient chiant.

@tryon-dev
Copy link
Contributor

est ce que l'interface ressemble au screen qui ont été envoyé précédemment ?

@tryon-dev
Copy link
Contributor

Sur certains points, oui, très clairement, notamment le manque de communication, point que j'ai souligné et rabaché et qui m'a coûté ma place au sein de l'équipe, et il n'y a pas d'amélioration.

Donc oui, à un moment, ça devient chiant.

Ca marche

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 16, 2024

@tryon-dev je mettrai des captures tout à l'heure, j'ai oublié d'en mettre comme elle était pas encore prête

@Louis-htmlcss
Copy link
Contributor

@tryon-dev je mettrai des captures tout à l'heure, j'ai oublié d'en mettre comme elle était pas encore prête

Encore un avantage a notre pr qui permet une preview expo

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 16, 2024

@tryon-dev j'ai importé des vidéos et des captures dans la description de la pr :)
Déso si les vidéos sont pas directement intégrés dans le tableau

@Louis-htmlcss
Copy link
Contributor

Ça fait quand même 20 jours

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 29, 2024

😭

@Clmnnt
Copy link
Contributor

Clmnnt commented Nov 30, 2024

oui personne review... (je dis ça j'ai pas fait sorry)

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 30, 2024

Si personne review juste pask expo 52 est obligatoire, alors faites une review de la pr #370 et faites un merged pour pouvoir avancer !

Ou sinon, télécharger un apk d'expo en version 2.31.xx et désinstaller la version actuelle pour Android. Et pour ceux qui sont sur ios, faites la commande npm run ios pour lancer un simulateur !

@Clmnnt
Copy link
Contributor

Clmnnt commented Nov 30, 2024

J'ai vu ça j'suis en train de review la PR sur Expo 52 du coup ;)

@Orafilynie
Copy link
Contributor

Si personne review juste pask expo 52 est obligatoire, alors faites une review de la pr #370 et faites un merged pour pouvoir avancer !

Ou sinon, télécharger un apk d'expo en version 2.31.xx et désinstaller la version actuelle pour Android. Et pour ceux qui sont sur ios, faites la commande npm run ios pour lancer un simulateur !

Euh la version expo 52 presente un bug un peut dérangeant sur la homepage sur build natif IOS (bug non present sur expo go)

ScreenRecording_11-30-2024.13-44-00_1.mp4

@Clmnnt
Copy link
Contributor

Clmnnt commented Nov 30, 2024

Si personne review juste pask expo 52 est obligatoire, alors faites une review de la pr #370 et faites un merged pour pouvoir avancer !
Ou sinon, télécharger un apk d'expo en version 2.31.xx et désinstaller la version actuelle pour Android. Et pour ceux qui sont sur ios, faites la commande npm run ios pour lancer un simulateur !

Euh la version expo 52 presente un bug un peut dérangeant sur la homepage sur build natif IOS (bug non present sur expo go)

ScreenRecording_11-30-2024.13-44-00_1.mp4

La question c'est maintenant comment on fix ça ? Possible, impossible ?

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 30, 2024

Si personne review juste pask expo 52 est obligatoire, alors faites une review de la pr #370 et faites un merged pour pouvoir avancer !

Ou sinon, télécharger un apk d'expo en version 2.31.xx et désinstaller la version actuelle pour Android. Et pour ceux qui sont sur ios, faites la commande npm run ios pour lancer un simulateur !

Euh la version expo 52 presente un bug un peut dérangeant sur la homepage sur build natif IOS (bug non present sur expo go)

ScreenRecording_11-30-2024.13-44-00_1.mp4

Transfère ce message sur la pr #372 stp

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Nov 30, 2024

Si personne review juste pask expo 52 est obligatoire, alors faites une review de la pr #370 et faites un merged pour pouvoir avancer !
Ou sinon, télécharger un apk d'expo en version 2.31.xx et désinstaller la version actuelle pour Android. Et pour ceux qui sont sur ios, faites la commande npm run ios pour lancer un simulateur !

Euh la version expo 52 presente un bug un peut dérangeant sur la homepage sur build natif IOS (bug non present sur expo go)

ScreenRecording_11-30-2024.13-44-00_1.mp4

La question c'est maintenant comment on fix ça ? Possible, impossible ?

Oui, voir ma pr #372 et dites de faire un revert de la nouvelle page d'accueil également sur ios

@godetremy
Copy link
Contributor

Oui, voir ma pr #372 et dites de faire un revert de la nouvelle page d'accueil également sur ios

Non, cette page ne sera pas revert ! Si vous la trouvez instable, alors on la debug !

Les informations concernant Expo52 seront bientôt public, mais on y travaille !

@Kgeek33
Copy link
Contributor Author

Kgeek33 commented Dec 10, 2024

@godetremy d'accord, j'y travaillerai sur ma pr #372 (va être en draft)
mais peux-tu review cette pr, je n'y touche pas à la page d'accueil stp ?

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.

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