-
Notifications
You must be signed in to change notification settings - Fork 66
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this 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é !
There was a problem hiding this 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"
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 |
En tout cas c'est déjà très propre le code |
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 ! |
Yes ce serait bien en effet, y'a quoi comme "bug de fou"? |
bien vu si tu pourrais les link à la PR ce serait cool @tom-theret |
C'est bon normalement |
Non rajoute Closes devant le numéro des PR pour les 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) |
Je le ferai en V2 |
Ce serait intéressant un widget moyenne générale en format petit moyen grand (pas obligatoire, je propose juste)
|
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. |
Ok Btw y'a ça sur le figma https://www.figma.com/design/EZ9HbmMmCywyCxp3OuvofF/%5BPUBLIC%5D-Papillon-Product-Design?node-id=3-6035&p=f pour le design des widgets |
🚀 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].
TODO
(aka des annotations pour du code manquant) dans vos modificationsChangelogs 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:
android/app/build.gradle
: UpdatedversionCode
to7700
andversionName
to7.7.0
.ios/Papillon/Info.plist
: UpdatedCFBundleShortVersionString
to7.7.0
.Entitlements:
app.json
: Addedcom.apple.security.application-groups
entitlement.ios/Papillon/Papillon.entitlements
: Addedcom.apple.security.application-groups
entitlement.ios/intent/intent.entitlements
: Created new entitlements file withcom.apple.security.application-groups
.Widget Implementation:
ios/intent/Info.plist
: Added a newInfo.plist
for intent handling withSelectAccountIntent
.ios/intent/IntentHandler.swift
: Implemented intent handling for selecting accounts and retrieving account data.ios/widget/News/NewsWidget.swift
: Added the main implementation for the news widget, including data retrieval, timeline provider, and widget configuration.ios/widget/News/View/systemMediumNews.swift
: Implemented the medium-sized news widget view.ios/widget/News/View/systemLargeNews.swift
: Implemented the large-sized news widget view.Informations supplémentaires
(les données présentées sont bien entendu pas réelles et tout une logique est implémentée )
#431
#153