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]: Ajout des widgets iOS #549

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

Conversation

tom-theret
Copy link
Contributor

@tom-theret tom-theret commented Dec 27, 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

This pull request includes a variety of changes to the Android and iOS project files, focusing on version updates, entitlements, and widget implementation. The most important changes include updating the app version, adding new entitlements, and implementing a new widget for displaying news.

Version Updates:

Entitlements:

Widget Implementation:

Informations supplémentaires

Simulator Screenshot - iPhone 16 Pro - 2024-12-27 at 18 32 32
Simulator Screenshot - iPhone 16 Pro - 2024-12-27 at 18 32 27

(les données présentées sont bien entendu pas réelles et tout une logique est implémentée )

#431
#153

@tom-theret tom-theret changed the title Ios/widget [FEAT]: Ajout des widgets iOS Dec 27, 2024
Copy link
Contributor

@raphckrman raphckrman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM côté Swift bien joué :) !
Pour le dernier screen que tu as mis, le widget fait un peu bizarre, un grand espace blanc pour marquer ensuite juste "Aucun cours, bon repos!" ça fait très vide tout en occupant beaucoup de place, sinon bien joué !

Copy link
Contributor

@JyhuKo JyhuKo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM mais je suis d'accord pour le widget "vide"

@yannouuuu yannouuuu added ✨ enhancement New feature or request 🎨 interface Element visible to the user labels Dec 27, 2024
@tom-theret
Copy link
Contributor Author

Je sais mais la c'est une V1 que je demande de merge, c'est pour que les loulous ai les widgets EDT et Actu à la rentrée, bien sûr il y a des axes d'amélioration. Mais, ça sera pour une V2 (le Swift c'est chiant)

@raphckrman
Copy link
Contributor

Je sais mais la c'est une V1 que je demande de merge, c'est pour que les loulous ai les widgets EDT et Actu à la rentrée, bien sûr il y a des axes d'amélioration. Mais, ça sera pour une V2 (le Swift c'est chiant)

Je suis en plein dans Swift actuellement je pourrais check plus tard si tu veux

@raphckrman
Copy link
Contributor

En tout cas c'est déjà très propre le code

@tom-theret
Copy link
Contributor Author

Merci, mais là, j'ai surtout besoins de m'occuper de l'app principal qui a des bugs de fou.

Puis j'aimerais faire une app Apple Watch !

@raphckrman
Copy link
Contributor

Yes ce serait bien en effet, y'a quoi comme "bug de fou"?

@Nicolasfr340
Copy link

Nicolasfr340 commented Dec 27, 2024

En relation avec les issues #431 #153 sur les widgets iOS

@raphckrman
Copy link
Contributor

bien vu si tu pourrais les link à la PR ce serait cool @tom-theret

@tom-theret
Copy link
Contributor Author

C'est bon normalement

@raphckrman
Copy link
Contributor

Non rajoute Closes devant le numéro des PR pour les link

@Nicolasfr340
Copy link

Ce serait intéressant pour le widget des cours de mettre le même texte au milieu que dans l'EDT (comme pour les vacances et la refonte de #535)

@tom-theret
Copy link
Contributor Author

Je le ferai en V2

@Nicolasfr340
Copy link

Ce serait intéressant un widget moyenne générale en format petit moyen grand (pas obligatoire, je propose juste)

  • Petit avec juste la moyenne générale perso et classe
  • Moyen avec moyenne générale perso et classe dans un affichage moins compacte, et aussi avec max théorique et mini théorique OU alors juste moyenne générale perso et classe (dans un affichage moins compacte donc)
  • Large même chose que moyen mais avec le graphique de progression et comparaison avec moyenne classe

@tom-theret
Copy link
Contributor Author

Je sais mais j'ai pas de compte avec des notes. Mais je pense créer un serveur qui donne du json qui simule une instance quelconque.

@Nicolasfr340
Copy link

@JyhuKo
Copy link
Contributor

JyhuKo commented Dec 28, 2024

aussi ici c est pas tres logique car c est censé etre le prochain cours mis ca met un cours d'il ya une sqemaine
image

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

Successfully merging this pull request may close these issues.

7 participants