Skip to content

Admin: Permettre de modifier la date de début de suivi d'un membre de groupe GPS - #5517

Merged
tonial merged 1 commit into
masterfrom
alaurent/gps_admin
Feb 3, 2025
Merged

Admin: Permettre de modifier la date de début de suivi d'un membre de groupe GPS#5517
tonial merged 1 commit into
masterfrom
alaurent/gps_admin

Conversation

@tonial

@tonial tonial commented Feb 1, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Pour permettre de donner une date logique de début de suivi aux groupes créés manuellement.
Vu la faible sensibilité de cette date, je l'ai rendue accessible tout le temps pour simplifier la fonctionnalité (pas besoin de ne le faire que pour la création et la modification de membership créées manuellement)

🍰 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 ?
  • Ajouter l'étiquette « Bug » ?

🏝️ 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 self-assigned this Feb 1, 2025
@tonial tonial added the modifié Modifié dans le changelog. label Feb 1, 2025
@tonial
tonial changed the base branch from master to alaurent/gps_sync_groups_backlog February 1, 2025 19:04
@tonial
tonial force-pushed the alaurent/gps_admin branch 2 times, most recently from 46136d4 to 1d45c9c Compare February 3, 2025 11:07
@tonial
tonial force-pushed the alaurent/gps_sync_groups_backlog branch 4 times, most recently from 17d3a56 to 58759d4 Compare February 3, 2025 11:18
@tonial
tonial force-pushed the alaurent/gps_admin branch 2 times, most recently from 69ad3a7 to 66341e5 Compare February 3, 2025 13:09
@tonial
tonial force-pushed the alaurent/gps_sync_groups_backlog branch from 58759d4 to ea7f953 Compare February 3, 2025 13:09
Base automatically changed from alaurent/gps_sync_groups_backlog to master February 3, 2025 13:28
@tonial
tonial force-pushed the alaurent/gps_admin branch from 66341e5 to 89f2d48 Compare February 3, 2025 13:30
Comment thread tests/gps/test_admin.py
Comment on lines +58 to +61
"created_at_0": "01/01/2025",
"created_at_1": "12:34:56",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Dans l'admin, ce n'est pas utile de vérifier qu'on retrouve ces infos après le POST ?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Si, c'est utile en effet, j'ai corrigé

@tonial
tonial force-pushed the alaurent/gps_admin branch from 89f2d48 to c88d7e9 Compare February 3, 2025 13:34
@tonial
tonial requested a review from EwenKorr February 3, 2025 13:35
@tonial
tonial added this pull request to the merge queue Feb 3, 2025
Merged via the queue into master with commit a3acb62 Feb 3, 2025
@tonial
tonial deleted the alaurent/gps_admin branch February 3, 2025 13:57
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.

2 participants