Fix(Webview): Suppression alerte et affichage directement lors du choix du service

[Kgeek33]

🚀 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

J'ai vu sur les commentaires du stores qu'il arrive que ça affiche la popup pour dire que seul les comptes élèves sont supportés alors qu'il ne sait même pas connecté ! Et effectivement, cela arrive rarement que la WebView s'arrête avant de faire une redirection sur l'ent et donc, l'affichage de la popup

• Celle ci a été supprimé
• Un avertissement en gras est désormais affiché dès qu'on ajoute un compte, sur le choix du service
• Optimisation du code (comme d'hab 😉)

Informations supplémentaires

L'affichage de l'avertissement :

Issue en lien

• Close [Bug]: EcoleDirecte : bug à la connexion Expected a value from the enum AccountKind but got '1' #588

[ecnivtwelve]

Je sais pas si l'avertissement en gras est forcément la meilleure option.
Concernant l'ajout d'un build number, je l'ai volontairement supprimée car Xcode et App Store Connect gère automatiquement celle-ci au moment de la signature du build.

[Kgeek33]

Je sais pas si l'avertissement en gras est forcément la meilleure option.
Concernant l'ajout d'un build number, je l'ai volontairement supprimée car Xcode et App Store Connect gère automatiquement celle-ci au moment de la signature du build.

Ok je viens de supprimer le gras, je savais pas si je devais mettre ou non
Ok je savais pas que XCode faisait ça, j'ai remis comme tu avais fait !

[ecnivtwelve]

Je parlais pas du gras en particulier mais de l'emplacement de l'alerte, à voir avec @toi-et-moi si ça pose problème

[Kgeek33]

alors @ecnivtwelve tu peux supprimer la popup le temps de la review pour éviter le bug pour la 7.7 ?
sauf si ça sera une review rapide

[ecnivtwelve]

Ouais mais en échange beaucoup de parents vont se questionner... Je pense qu'on va pousser ta pr sur un patch de v7.7.1

[Kgeek33]

Ouais mais en échange beaucoup de parents vont se questionner... Je pense qu'on va pousser ta pr sur un patch de v7.7.1

oui c'est ça le problème, un chargement infini lors de la connexion bof -_-
ok très bien, le bug se produit rarement donc c'est pas urgent