Skip to content

Fiche de poste : Afficher le tableau des fiches de poste même s'il ne contient que la ligne des candidatures spontanées - #6619

Merged
EwenKorr merged 1 commit into
masterfrom
ewen/depub
Aug 1, 2025
Merged

Fiche de poste : Afficher le tableau des fiches de poste même s'il ne contient que la ligne des candidatures spontanées#6619
EwenKorr merged 1 commit into
masterfrom
ewen/depub

Conversation

@EwenKorr

@EwenKorr EwenKorr commented Jul 28, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Le tableau n'est jamais vide, il y a forcément au moins la ligne des candidatures spontanées.

Et cela rend plus logique la présence du bandeau dans le cas où il n'y a que l'entrée des candidatures spontanées, non mise à jour depuis au moins 2 mois (voir captures plus bas).

🍰 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

Avant :

image

Après :

image

@EwenKorr EwenKorr added the modifié Modifié dans le changelog. label Jul 28, 2025
@EwenKorr EwenKorr self-assigned this Jul 28, 2025
@EwenKorr
EwenKorr requested a review from leo-naeka July 28, 2025 13:41
@notion-workspace

Copy link
Copy Markdown

@leo-naeka leo-naeka left a comment

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.

LGTM, mais j'en aurais bien profité pour s'assurer de la présence de la ligne dans TestJobDescriptionListView.test_response_content (ou bien une nouvelle TestJobDescriptionListView.test_response_content_with_no_job_descriptions)

Even if there is no job description, it is not empty because of the
spontaneous application row.
@EwenKorr

Copy link
Copy Markdown
Contributor Author

Ça faisait effectivement vide.

</tr>
</thead>
<tbody>
{# Special row for controlling spontaneous applications #}

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.

Ça me fait me demander si cette ligne ne devrait pas être uniquement présente sur la page 1 ?

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.

Je m'en occupe dans une autre PR.
Mais avant je corrige/consolide les include "includes/pagination.html" (dans les tests, quand on a suffisamment d'éléments pour afficher la pagination, on a souvent des erreurs de variable non reconnue)

@EwenKorr
EwenKorr added this pull request to the merge queue Aug 1, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 1, 2025
@EwenKorr
EwenKorr added this pull request to the merge queue Aug 1, 2025
Merged via the queue into master with commit e25f401 Aug 1, 2025
14 checks passed
@EwenKorr
EwenKorr deleted the ewen/depub branch August 1, 2025 09:35
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