Skip to content

Conversation

@tonial
Copy link
Contributor

@tonial tonial commented Jul 9, 2024

🤔 Pourquoi ?

Améliorer les données affichées aux utilisateurs de GPS

🍰 Comment ?

Décrivez en quelques mots la solution retenue et mise en oeuvre, les difficultés ou problèmes rencontrés. Attirez l'attention sur les décisions d'architecture ou de conception importantes.

🚨 À vérifier

  • Mettre à jour le CHANGELOG_breaking_changes.md ?

🏝️ Comment tester

Les instructions pour reproduire le problème, les profils de test, le parcours spécifique à utiliser, etc. Si vous disposez d'une recette jetable, mettre l'URL pour tester dans cette partie.

💻 Captures d'écran

@tonial tonial requested a review from celine-m-s July 9, 2024 12:33
@tonial tonial self-assigned this Jul 9, 2024
@tonial tonial force-pushed the alaurent/gps_update_created_at branch from 401d7c5 to e3a44d9 Compare July 9, 2024 12:34
@tonial tonial added the modifié Modifié dans le changelog. label Jul 9, 2024
@tonial tonial force-pushed the alaurent/gps_update_created_at branch 4 times, most recently from b190f98 to d0861d0 Compare July 9, 2024 13:00
@tonial tonial force-pushed the alaurent/gps_update_created_at branch 2 times, most recently from c320c5c to 5fb2175 Compare July 9, 2024 14:00
Copy link
Contributor

@celine-m-s celine-m-s left a comment

Choose a reason for hiding this comment

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

Toooop !
Si tu as bien testé et que ça te semble cohérent, c'est bon pour moi. Sinon, je serais rassurée d'avoir un petit test supplémentaire. ^_^
À toi de voir.

@tonial
Copy link
Contributor Author

tonial commented Jul 9, 2024

J'ai testé en local 🙈

Je peux écrire un test pour vérifier que ça fait ce qu'on veut, rendez-vous en fin de semaine

@celine-m-s
Copy link
Contributor

Je peux écrire un test pour vérifier que ça fait ce qu'on veut, rendez-vous en fin de semaine

Pourquoi en fin de semaine ?

Si c'est bon pour toi, c'est parti ! Moi je ne peux pas tester.

@tonial tonial force-pushed the alaurent/gps_update_created_at branch from 5fb2175 to 818e702 Compare July 11, 2024 08:15
@tonial
Copy link
Contributor Author

tonial commented Jul 11, 2024

Test ajouté ce qui m'a permis de modifier la commande init pour mettre created_in_bulk=True (just in case)

@tonial tonial added this pull request to the merge queue Jul 11, 2024
Merged via the queue into master with commit f28bb7e Jul 11, 2024
@tonial tonial deleted the alaurent/gps_update_created_at branch July 11, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

modifié Modifié dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants