Minor changes + Alerte déconnexion

[Slysoks]

🚀 Nouvelle Pull Request

Checklist d'avant pull request

• 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

• Ajout d'un https:// devant l'URL de connexion si https:// ou http:// ne sont pas précisés
• Changement de formule d'appel de l'utilisateur (vouvoiement -> tutoiement)
• Nombre de lignes variables dans l'infobulle
• Ajout d'une belle alerte de déconnexion

Informations supplémentaires

• Passage de l'URL des CGU et ToS dans des variables pour simplifier leur édition en cas de besoin
• Affichage de l'os et de sa version en dessous de la version du programme
• Ajout d'un lien pour voir les collaborateurs de Papillon

[Slysoks]

Ca fait peut-être un peu bcp là mais ça vaut le coup je pense !!

[Gabriel29306]

Tu pourrais envoyer un screen shot de l'alerte de déconnexion ?

[Slysoks]

[Slysoks]

Et voilà

[Gabriel29306]

LGTM

[Slysoks]

Et bonne rentrée à toute l'équipe (car je pense que vous aussi c'est mardi) !

[Rexxt]

nan ma pré c'était ce matin, premier cours dans 1sem, mais merci du coup

[Rexxt]

par rapport à l'alerte elle est un poil déséquilibrée visuellement mais ça se règle

[tom-theret]

Il y a une alert prévu à cette effet et même pour l'entièreté de d'app mais n'est pas documenté.

[tom-theret]

Ne pas utiliser d'alert custom alors qu'on en a une qui a été spécifiquement créer pour ce type d'occasion.

[Slysoks]

La quelle ?

[Slysoks]

Car si c'est celle dans le @/src/providers/AlertProviders, elle ne convient pas au style de l'application

[tom-theret]

Car si c'est celle dans le @/src/providers/AlertProviders, elle ne convient pas au style de l'application

On peut la remixer pour qu'elle supporte n'importe quel situation !

[Slysoks]

Je vous fait ça demain !

[Slysoks]

En fait, je me suis très vite rappelé que je ne comprends pas vraiment le coté Type de TypeScript (car je suis habitué au JS), donc j'ai tout remis comme avant au niveau de cette notif...

[godetremy]

Perso, je trouve qu'il y a beaucoup de motifs pour une seul PR, si tu pouvais la séparer en plusieurs ça serait super !

[Slysoks]

Jpp

[Louis454545]

Jpp

Mdrrrr

[godetremy]

Jpp

Tant pis je le ferai...

[tryon-dev]

Je close car finalement quelqu'un d'autre la fait plus intégré a l'ui design de papillon

[Slysoks]

Je close car finalement quelqu'un d'autre la fait plus intégré a l'ui design de papillon

Tkt, j'en referai une pr les "Minor changes" uniquement