Skip to content

fix(Homeworks): disable toggleHomeworkState for Skolengo #848

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

Merged
merged 2 commits into from
Mar 22, 2025

Conversation

Kgeek33
Copy link
Contributor

@Kgeek33 Kgeek33 commented Mar 20, 2025

✨ Nouvelle Pull Request

Merci de contribuer à l'amélioration de Papillon !

Tu te poses des questions sur les pull requests (PR) ? Une documentation a spécialement été créée ici => https://gitbook.getpapillon.xyz/organisation/outils-internes/github

Avant toute chose...

Tu t'assures avoir respecté les points suivants (en rajoutant un x dans les crochets) :

  • 📲 J'ai testé l'application via Expo Go et/ou Build et elle fonctionne correctement
  • 🗣️ J'utilise le langage informel (tutoiement)
  • 📃 Cette PR n'est pas un duplicata et est prête à être review et merge
  • ❌ Je n'ai pas fait d'erreurs TypeScript et ESLint dans le code
  • 📝 J'ai fait une description des changement effectués ci-dessous

Résumé des changements effectués

Note

Je ne suis pas sûr que ça fonctionne mais en regardant le code, c'est sûrement le problème potentiel

La fonction toggleHomeworkState n'est plus appelée sur les comptes Skolengo, renvoyant une erreur et (possiblement) arrête la suite du code

Capture(s) d'écran (pour rendre le test de ta PR rapide)

undefined

Issue(s) en rapport

Note

Cette section te permet de référencer des issues en lien avec ta PR. Pour que les issues soient fermées automatiquement au merge, utilise les "closing keywords" de Github (exemples ci-dessous).

S'il y en a plusieurs, continuer à les lister

@tryon-dev tryon-dev merged commit 34ac52f into PapillonApp:main Mar 22, 2025
@Kgeek33 Kgeek33 deleted the fix/skolengo branch March 22, 2025 17:26
Kgeek33 added a commit to Kgeek33/Papillonv7 that referenced this pull request Mar 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants