Preview expo

[Louis-htmlcss]

🚀 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].

• [x] Vous avez testé de build le projet avec vos modifications et ce build a réussi
• [x] Vous respectez les conventions de codage et de nommage du projet
• [x] Vous utilisez la tabulation pour l'indentation afin de maintenir un code lisible
• [x] Cette pull request n'est pas un duplicata d'une autre
• [x] Cette pull request est prête à être revue (review) et fusionnée (merge)
• [x] Il n'y a pas de TODO (aka des annotations pour du code manquant) dans vos modifications
• [x] Il n'y a pas d'erreurs de langue dans votre code (grammaire, vocabulaire, conjugaison, orthographe)
• [x] Les détails des changements ont été décrits ci-dessous
• [x] 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

Voici le workflow incroyable qui permet de toujours avoir des preview expo à portée de main.

Avec @Kgeek33 (le goat), on a fait ce workflow au petit oignon pour pouvoir tester, debugger et revoir facilement les pr sans regarder le code (même s’il faut quand même le faire…).

Bref, comme ça, on enlève les systèmes beaucoup trop complexes, comme l'avait suggéré @godetremy de refaire tout le fonctionnement de Expo.

Puis il n'y a pas que ça ! @Kgeek33 a fait ce nouveau template de PR avec beaucoup moins de choses à cocher et avec plus d'informations pour review plus rapidement !

La partie bot serait réservée pour la preview Expo et Sourcery Ai, je sais que vous n'aviez pas voulu de ça avant, mais je trouve ça vraiment important, surtout que certaine personne ne mette pas leur changement dans leur description de pr ! Alors maintenant sourcery ai peut aussi le faire ! Et aussi avoir une mini review super rapide ! Par ailleurs elle permet de link les issue concerné ce qui permet de fermer les issue beaucoup plus rapidement !

Le bot deepsource sera aussi d'une grande aide précieuse pour aider a écrit tout simplement du meilleur code.

Aussi, à chaque PR, un build sera lancé pour Android et iOS pour qu'on sache s'il n'y a pas d'erreur !

Merci de review vite pour utiliser cette fonctionnalité dans toutes les pull request !

[Louis-htmlcss]

This pull request includes significant updates to the GitHub workflows and the pull request template to improve the development process and streamline contributions. The most important changes include the creation of a new workflow for generating QR codes for pull requests, updates to the pull request template to enhance clarity and usability, and modifications to existing workflows to trigger on pull request events.

Workflow Enhancements:

• .github/workflows/expo_preview.yml: Introduced a new workflow to generate QR codes for pull requests, which includes steps for checking out the PR, setting up the environment, starting the Expo server, and updating the PR description with a QR code link.

• .github/workflows/start_build.yml: Updated the workflow to trigger on pull request events and added a condition to the deploy_to_beta_channels job to run only when the workflow is manually dispatched. [1] [2]

Documentation and Template Improvements:

• .github/PULL_REQUEST_TEMPLATE.md: Revised the pull request template to improve clarity, provide better guidance on the contribution process, and include sections for summarizing changes, linking related issues, and adding screenshots.

[Kgeek33]

Cette pr est en draft, car elle n'est pas encore complète D'autres changements (tel que la simplification du remplissage des issues, une optimisation et amélioration des workflow...) sont à venir

À partir de demain, je pourrai m'y mettre avec @Louis-htmlcss :)

[Louis-htmlcss]

@yannouuuu

[Louis-htmlcss]

@Kgeek33 bro ya mon repo au lieu de papillon dans le truc de sécurité 💀

[Kgeek33]

@Kgeek33 bro ya mon repo au lieu de papillon dans le truc de sécurité 💀

oh l'erreur mdrrr je corrige ça

[godetremy]

Je trouve que faire tourner plusieurs serveur expo ce n'est pas franchement la meilleur solution, même si ce n'est pas précisé j'estime que GH limite le nombre d'actions en simultané, donc perso je suis pas sur de l'idée

[Kgeek33]

Alors je pensais à faire une mention (par exemple #build), et quand github actions détecte cette mention, ça lance le qr code pour une durée d'1 heure et après ça s'arrête

[Louis-htmlcss]

Je trouve que faire tourner plusieurs serveur expo ce n'est pas franchement la meilleur solution, même si ce n'est pas précisé j'estime que GH limite le nombre d'actions en simultané, donc perso je suis pas sur de l'idée

Ya pas écrit donc blc il avait qu'à le marquer

[Louis-htmlcss]

Merci de me poing si tu me réponds @godetremy

[godetremy]

Alors je pensais à faire une mention (par exemple #build), et quand github actions détecte cette mention, ça lance le qr code pour une durée d'1 heure et après ça s'arrête

C'est bien la PIRE IDÉE que j'ai jamais vu ! Ça fait que remplir les discussions inutilement, ça envoie des mails a tout les reviewer, et GitHub c'est pas Discord c'est tout !

[godetremy]

Ya pas écrit donc blc il avait qu'à le marquer

@Louis-htmlcss C'est pas parce qu'il ne l'écrivent pas qu'il ne faut pas y penser ! Un ban de notre repo sur GitHub c'est possible donc solo !

[Louis-htmlcss]

Ya pas écrit donc blc il avait qu'à le marquer

@Louis-htmlcss C'est pas parce qu'il ne l'écrivent pas qu'il ne faut pas y penser ! Un ban de notre repo sur GitHub c'est possible donc solo !

Dans tt les cas il ya exactement le même nombre de workflow run que de pr donc je trouve ça en fait pas énormément

[Kgeek33]

Ya pas écrit donc blc il avait qu'à le marquer

@Louis-htmlcss C'est pas parce qu'il ne l'écrivent pas qu'il ne faut pas y penser ! Un ban de notre repo sur GitHub c'est possible donc solo !

Oui mais du coup, par rapport à ton idée principale, le serveur que tu voulais mettre en place, c'est pas du github actions en continu ?

[Louis-htmlcss]

Ya pas écrit donc blc il avait qu'à le marquer

@Louis-htmlcss C'est pas parce qu'il ne l'écrivent pas qu'il ne faut pas y penser ! Un ban de notre repo sur GitHub c'est possible donc solo !

Oui mais du coup, par rapport à ton idée principale, le serveur que tu voulais mettre en place, c'est pas du github actions en continu ?

Non, je crois il voulait l host sur un autre serveur mais ici ya un autre problème celui de sécurité, car il a tout les logs de expo

[Louis-htmlcss]

Et où modifications du bundle

[Louis-htmlcss]

Alors je pensais à faire une mention (par exemple #build), et quand github actions détecte cette mention, ça lance le qr code pour une durée d'1 heure et après ça s'arrête

C'est bien la PIRE IDÉE que j'ai jamais vu ! Ça fait que remplir les discussions inutilement, ça envoie des mails a tout les reviewer, et GitHub c'est pas Discord c'est tout !

Complètement débile car vous pouvez désactiver les notifs du workflow

[Louis-htmlcss]

Ya pas écrit donc blc il avait qu'à le marquer

@Louis-htmlcss C'est pas parce qu'il ne l'écrivent pas qu'il ne faut pas y penser ! Un ban de notre repo sur GitHub c'est possible donc solo !

Et puis si ça vous tellement peut de vous faire ban vous pouvez le lancer sur un autre repo

[Louis-htmlcss]

Je trouve que faire tourner plusieurs serveur expo ce n'est pas franchement la meilleur solution, même si ce n'est pas précisé j'estime que GH limite le nombre d'actions en simultané, donc perso je suis pas sur de l'idée

Et aussi, merci d'être un peu plus respectueux des idées des autres @Kgeek33

[Kgeek33]

Je trouve que faire tourner plusieurs serveur expo ce n'est pas franchement la meilleur solution, même si ce n'est pas précisé j'estime que GH limite le nombre d'actions en simultané, donc perso je suis pas sur de l'idée

Et aussi, merci d'être un peu plus respectueux des idées des autres @Kgeek33

Tu t'adresses à moi là ??? J'ai critiqué en quoi les idées de qqun là ?!!

[Louis-htmlcss]

Je trouve que faire tourner plusieurs serveur expo ce n'est pas franchement la meilleur solution, même si ce n'est pas précisé j'estime que GH limite le nombre d'actions en simultané, donc perso je suis pas sur de l'idée

Et aussi, merci d'être un peu plus respectueux des idées des autres @Kgeek33

Tu t'adresses à moi là ??? J'ai critiqué en quoi les idées de qqun là ?!!

Je parle à @godetremy pas à toi mdrr

[Kgeek33]

Je trouve que faire tourner plusieurs serveur expo ce n'est pas franchement la meilleur solution, même si ce n'est pas précisé j'estime que GH limite le nombre d'actions en simultané, donc perso je suis pas sur de l'idée

Et aussi, merci d'être un peu plus respectueux des idées des autres @Kgeek33

Tu t'adresses à moi là ??? J'ai critiqué en quoi les idées de qqun là ?!!

Je parle à @godetremy pas à toi mdrr

Ah bon mdrr

[godetremy]

Dans tt les cas il ya exactement le même nombre de workflow run que de pr donc je trouve ça en fait pas énormément

Peut importe le nombre de workflow, on est pas la pour surcharger les serveur de GitHub avec des actions aussi stupide que celle-ci

Oui mais du coup, par rapport à ton idée principale, le serveur que tu voulais mettre en place, c'est pas du github actions en continu ?

Non, sur ma PR, l'action sert uniquement à compiler le code vers un bundle expo et la mettre sur un autre repos ou serveur. Ensuite mon serveur sert à ajouter les meta d'Expo.

Non, je crois il voulait l host sur un autre serveur mais ici ya un autre problème celui de sécurité, car il a tout les logs de expo

Non, les logs sont fait sur l'appareil, mais mon serveur ne les reçois pas car je n'ai pas fait le tunnel qui permet de lier les appareils. Je n'ai accès qu'à votre adresse IP (qui me permet de lier les assets au bon devices), et au assets que tu souhaite consulter. Toute erreurs ou autre sont géré par ton appareil. Le code est disponible ici.

Complètement débile car vous pouvez désactiver les notifs du workflow

Les discussions dans les PR sa doit uniquement parler du code. Ce n'est pas discord ! On ne parle pas uniquement des notifications du workflow mais des mail de discussion ! Chaque jour, je dois supprimer mes mails où je suis mentionné car vous parler sans cesses. S'en ai vraiment lassant.

Donc hors ce qui est des discussions ou question réponse à propos du code, direction Discord !

Et puis si ça vous tellement peur de vous faire ban vous pouvez le lancer sur un autre repo

Si la police t'arrête pour meurtre, tu prend de la prison, après tu continue toi ? Même principe !

Et aussi, merci d'être un peu plus respectueux des idées des autres

Je suis respectueux, je ne donne que mon point de vu, mais on est quand même bon nombre à partager le même avis. Vince à dit hier dans notre groupe perso :

logiciel libre ne FORCE PAS l’ouverture à la contribution et on l’oublie un peu ça il y a bien liberté de réutiliser, modifier et redistribuer mais personne ne dit que cela doit être sur le même canal de distribution

N'oublions pas que Papillon est un client né d'une volonté d'expérience utilisateur, ou celle ci doit primer sur la fonctionnalité : des problèmes UI/UX doivent être rédhibitoires et prioritaires sur chaque PR

Rare dans le logiciel libre car on sait très bien ce qu'il se passe quand la communauté prime sur la forme : on accorde plus d'importance aux fonctionnalités et utilisateurs qu'a l'essentiel selon moi, l'expérience utilisateur

Papillon doit évidemment rester open source et libre, c'est le garant de la sécurité de nos données et l'accès au code pour tous : cependant, il nait et existe d'une vision, que l'on perd peu a peu plus on essaye de se plier aux demande des contribs

[tom-theret]

Points sur les querelles sociales

Les discussions dans les PR sa doit uniquement parler du code. Ce n'est pas discord ! On ne parle pas uniquement des notifications du workflow mais des mail de discussion ! Chaque jour, je dois supprimer mes mails où je suis mentionné car vous parler sans cesses. S'en ai vraiment lassant.

Je suis d'accord avec toi sur la fonction primaire des pull requests, mais il y quand même un problème chez Papillon qui est l'organisation encore et toujours, vous recevez des PR pour tout plein de fonctions, pourquoi ne pas faire un GH Project qui permet de recenser les idées que vous avez plus celle des contributeurs, d'en discuter puis de les développer ?

Papillon doit évidemment rester open source et libre, c'est le garant de la sécurité de nos données et l'accès au code pour tous : cependant, il nait et existe d'une vision, que l'on perd peu a peu plus on essaye de se plier aux demande des contributeurs

Encore une fois, c'est un projet qui est communautaire, vous avez des idées ? Faite nous part de ça ou alors développer vos idées dans votre coin comme tout le monde puis faite une PR comme tout le monde... et la communauté dira ce qu'elle pense de vos idées.

Je pense réellement qu'un GH Project qu'on automatise pour les issues, et où l'on rentre nos idées serait une bonne chose pour tout le monde. Papillon commence à se transformer en iTunes UI alors que toute les fonctions de Papillon sont très pertinentes.

Mais n'oublier pas qu'on s'adresse aussi à un public pas forcément initié des applications donc il faut vraiment faire du ménage avec l'UI et la rendre la plus pertinente.

Point sur les Workflows pour les previews

On peut se renseigner pour utiliser Snack (snack.expo.dev) qui permet de host du code en ligne et gratuitement surtout !

Puis pour le PR avec des fonctions 'native' alors créer un workflow de build classique qui nous permettra d'installer les apps sur nos simulateurs.

[Kgeek33]

@godetremy ok je comprends mieux pour ton action, merci. Du coup, que manque-t-il dans ton fichier yml pour qu'il soit merged ? @tom-theret Snack peut être très intéressant. Mais ça veut dire que pour chaque pr, il faudra aller sur Snack, mettre à jour le code (par exemple, le même que la branche main) et tester ? Ou alors on fait le PR, on met le même code sur Snack et on met sur la description de la PR le qr code généré par snack ?

[tom-theret]

@Kgeek33, en gros tu as une api qui te permet de créer des espaces, donc 1 PR = 1 Espace. Une fois ça fait on récupère les QR code pour Android et iOS si différents, on les affiches...

[Kgeek33]

@Kgeek33, en gros tu as une api qui te permet de créer des espaces, donc 1 PR = 1 Espace. Une fois ça fait on récupère les QR code pour Android et iOS si différents, on les affiches...

oh c'est cool ça a l'air intéressant ! Faut se renseigner

[Louis-htmlcss]

Non, sur ma PR, l'action sert uniquement à compiler le code vers un bundle expo et la mettre sur un autre repos ou serveur. Ensuite mon serveur sert à ajouter les meta d'Expo.

Non, je crois il voulait l host sur un autre serveur mais ici ya un autre problème celui de sécurité, car il a tout les logs de expo

Non, les logs sont fait sur l'appareil, mais mon serveur ne les reçois pas car je n'ai pas fait le tunnel qui permet de lier les appareils. Je n'ai accès qu'à votre adresse IP (qui me permet de lier les assets au bon devices), et au assets que tu souhaite consulter. Toute erreurs ou autre sont géré par ton appareil. Le code est disponible ici.

Ah, j'avais oublié que ta pr marchait comme ça, my bad, mais il y a toujours un problème où tu n'as pas répondu : tu peux modifier le bundle à ta guise, donc, pourvoir modifier l'application qu'on.

Complètement débile car vous pouvez désactiver les notifs du workflow

Les discussions dans les PR sa doit uniquement parler du code. Ce n'est pas discord ! On ne parle pas uniquement des notifications du workflow mais des mail de discussion ! Chaque jour, je dois supprimer mes mails où je suis mentionné car vous parler sans cesses. S'en ai vraiment lassant. Donc hors ce qui est des discussions ou question réponse à propos du code, direction Discord !

Je ne vois pas le rapport avec ma réponse. Car ici, le problème sont les messages des autres PRS et pas de mon workflow.

Et aussi, merci d'être un peu plus respectueux des idées des autres

Je suis respectueux, je ne donne que mon point de vu, mais on est quand même bon nombre à partager le même avis. Vince à dit hier dans notre groupe perso :

Pour moi, le message : « Pire idée » n'est pas une forme extrêmement respectueuse.