Skip to content

Admin : Ajout du reliquat pour les PASS IAE et de la durée pour les suspensions et prolongations#4147

Merged
rsebille merged 4 commits intomasterfrom
rsebille/admin-relicat
Jun 4, 2024
Merged

Admin : Ajout du reliquat pour les PASS IAE et de la durée pour les suspensions et prolongations#4147
rsebille merged 4 commits intomasterfrom
rsebille/admin-relicat

Conversation

@rsebille
Copy link
Contributor

🤔 Pourquoi ?

Simplifier la vie du support en n'ayant moins besoin de faire des calculs de dates à la main.

🍰 Comment ?

Champs supplémentaires dans le ModelAdmin et les Inlines.

💻 Captures d'écran

image
image

🏝️ Comment tester

  • Se connecter sur l'admin
  • Regarder un ou plusieurs PASS IAE ayant des suspensions/prolongations

@rsebille rsebille requested review from tonial and xavfernandez May 29, 2024 12:32
@rsebille rsebille self-assigned this May 29, 2024
Copy link
Contributor

@xavfernandez xavfernandez left a comment

Choose a reason for hiding this comment

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

Top la réorganisation des champs 👍


@admin.display(description="Durée")
def duration(self, obj):
return f"{obj.duration.days} jours"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
return f"{obj.duration.days} jours"
return f"{obj.duration.days} jour{pluralizefr(obj.duration.days}"

? 👼
Et pareil pour les autres ?

@rsebille rsebille force-pushed the rsebille/admin-relicat branch from ae7b3cf to 2f574bf Compare June 4, 2024 10:35
@rsebille rsebille added the ajouté Ajouté dans le changelog. label Jun 4, 2024
@rsebille rsebille added this pull request to the merge queue Jun 4, 2024
Merged via the queue into master with commit f03e2ea Jun 4, 2024
@rsebille rsebille deleted the rsebille/admin-relicat branch June 4, 2024 10:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants