Skip to content

Admin : Correction de l’affichage de l’état des candidatures contrôlée a posteriori#4768

Merged
francoisfreitag merged 1 commit intomasterfrom
ff/admin
Sep 18, 2024
Merged

Admin : Correction de l’affichage de l’état des candidatures contrôlée a posteriori#4768
francoisfreitag merged 1 commit intomasterfrom
ff/admin

Conversation

@francoisfreitag
Copy link
Member

🤔 Pourquoi ?

Affichait State: -.

🍰 Comment ?

L’attribut state n’existe pas sur les instances.
Django masque ces erreurs pour afficher un -:
https://github.com/django/django/blob/8e68f938f376cf2ca22a7e8ff0bcbe1b7a5832d1/django/contrib/admin/helpers.py#L269-L272

Previously, displayed `State: -`. The `state` attribute does not exist
on the evaluated job application instance. Django silences these errors
to display a `-`:
https://github.com/django/django/blob/8e68f938f376cf2ca22a7e8ff0bcbe1b7a5832d1/django/contrib/admin/helpers.py#L269-L272
@francoisfreitag francoisfreitag added the modifié Modifié dans le changelog. label Sep 18, 2024
@francoisfreitag francoisfreitag self-assigned this Sep 18, 2024
@francoisfreitag francoisfreitag added this pull request to the merge queue Sep 18, 2024
Merged via the queue into master with commit 9ef23b0 Sep 18, 2024
@francoisfreitag francoisfreitag deleted the ff/admin branch September 18, 2024 09:00
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