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

Com 3870 #694

Merged
merged 8 commits into from
Dec 19, 2024
Merged

Com 3870 #694

merged 8 commits into from
Dec 19, 2024

Conversation

ulysseferreira
Copy link
Collaborator

TESTS 💻

  • J'ai testé sur iphone :
    • simulateur
    • physique
  • J'ai testé sur android :
    • simulateur
    • physique
  • J'ai testé sur navigateur

POINTS D'ATTENTION POUR CETTE PR ⚠️

  • J'ai ajouté une variable d'environnement
  • Mes changements entrainent une incompatibilité avec l'ancienne version de l'api
    • Si oui, j'ai noté dans le Doc de déploiement qu'il faut fusionner l'api dans staging avant d'envoyer le build et déprécier les anciennes versions

Si tu as lu cette description, pense à réagir avec un 👁️

@ulysseferreira ulysseferreira added the bug Something isn't working label Dec 12, 2024
@ulysseferreira ulysseferreira self-assigned this Dec 12, 2024
@manonpalin
Copy link
Collaborator

manonpalin commented Dec 17, 2024

Comment as-tu defini quel composant definir dans un React.memo ? (pourquoi pas definir la pluspart si on sait que les props ne changent pas trop ?)

Ulysse : j'ai mis ceux sur lesquels on itérait pleins de fois et qui du coup sont rendus pleins de fois

Manon: Ok, je trouve que la page "A propos" d'uine formation qui a beaucoup de creneau est plutot lente aussi. Est-ce qu'on pourrait pas ameliorer les perfs aussi ?

Ulysse: j'ai essayé de gratter un peu, tu me diras si c'est mieux. J'ai ouvert une PR back aussi

@manonpalin manonpalin merged commit 88ed7cf into dev Dec 19, 2024
3 checks passed
@manonpalin manonpalin deleted the COM-3870 branch December 19, 2024 10:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants