Skip to content

Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées [GEN-2498]#6267

Merged
EwenKorr merged 2 commits into
masterfrom
ewen/employer_banner
Jun 6, 2025
Merged

Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées [GEN-2498]#6267
EwenKorr merged 2 commits into
masterfrom
ewen/employer_banner

Conversation

@EwenKorr

@EwenKorr EwenKorr commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

🤔 Pourquoi ?

Bientôt, les fiches de postes seront automatiquement dépubliées si elles ne sont pas mises à jour.
Il reste encore beaucoup de fiches non mises à jour.

On affiche un bandeau s'il existe au moins une fiche de poste qui n'a pas été mise à jour depuis au moins 60 jours.

@EwenKorr EwenKorr added the ajouté Ajouté dans le changelog. label Jun 5, 2025
@EwenKorr EwenKorr self-assigned this Jun 5, 2025
@EwenKorr EwenKorr added the ajouté Ajouté dans le changelog. label Jun 5, 2025
@notion-workspace

Copy link
Copy Markdown

@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch from ab9a558 to d09b935 Compare June 5, 2025 10:06
Comment thread itou/templates/layout/base.html Outdated
<strong class="me-2">Le service d’assistance sera indisponible les 30/05 et 04/06 pour des raisons techniques.</strong>
</div>
{# TODO: when deleting this banner, delete Company.has_active_job_descriptions too #}
{% if request.user.is_employer and request.current_organization and request.current_organization.has_active_job_descriptions %}

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.

On ne veut pas le mettre que pour celles qui n'ont pas été mises à jour depuis 3 mois ?

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.

Ce serait trop tard justement, l'idée est de prévenir les employeurs que s'ils ne mettent pas à jour, les vieilles offres seront dépubliées.

Il y a encore un gros paquet d'offres non actualisées, on a pas envie de dépublier la moitié des offres d'un coup 😬

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.

Je veux dire qu'il n'est pas utile d'affiche ce bandeau si toutes les fiches de postes actives de la structure ont été mises à jour il y a moins de 3 mois.
On veut cibler les structures qui ne l'ont pas encore fait car elles n'ont pas lu le mail

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.

Ah pardon
Pourquoi pas, je vais proposer au métier.

Après, pour les employeurs, on fait des requêtes pour afficher ce bandeau sur toutes les vues, j'imagine qu'il faut faire le plus simple possible. Enfirn, un WHERE en plus ne devrait pas faire trop de mal.

@EwenKorr EwenKorr added the 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC label Jun 5, 2025
@EwenKorr EwenKorr requested a review from hellodeloo June 5, 2025 10:28
@EwenKorr

EwenKorr commented Jun 5, 2025

Copy link
Copy Markdown
Contributor Author

C'est peut-être pas hyper joli le bouton en fin de ligne, tu aurais des suggestions @hellodeloo ?
image

@hellodeloo

hellodeloo commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

C'est peut-être pas hyper joli le bouton en fin de ligne, tu aurais des suggestions @hellodeloo ?

Oui, je te push un petit commit ui en debut d'aprem

@hellodeloo hellodeloo 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.

@EwenKorr Je t'ai push un petit correctif pour la bannière. Tu as pris la bannière "compte démo" en rèf mais c'est plutôt une bannière de ce type qu'il fait prendre en exemple. Texte sur une seule ligne (court si possible) et lien en texte.
capture 2025-06-05 à 14 31 08

@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch 4 times, most recently from a683656 to 4cbc753 Compare June 5, 2025 14:53
@EwenKorr EwenKorr added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Jun 5, 2025
@hellodeloo hellodeloo self-requested a review June 5, 2025 14:57
@EwenKorr

EwenKorr commented Jun 5, 2025

Copy link
Copy Markdown
Contributor Author

Je t'ai push un petit correctif pour la bannière. Tu as pris la bannière "compte démo" en rèf mais c'est plutôt une bannière de ce type qu'il fait prendre en exemple. Texte sur une seule ligne (court si possible) et lien en texte.

Merci beaucoup !

@github-actions

github-actions Bot commented Jun 5, 2025

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch 2 times, most recently from 7fb2062 to 1483466 Compare June 6, 2025 08:29
@EwenKorr EwenKorr changed the title Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non mises à jour [GEN-2498] Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées depuis 60 plus [GEN-2498] Jun 6, 2025
@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch from 1483466 to c325b6c Compare June 6, 2025 08:38
Comment thread itou/companies/models.py Outdated
Comment thread tests/companies/test_models.py Outdated
Comment thread tests/companies/test_models.py Outdated
Comment thread itou/companies/models.py Outdated

@rsebille rsebille 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.

Vu qu'a priori c'est temporaire, les commentaires sont plus là pour le partage que des changements à faire je dirais ;).

Comment thread itou/companies/models.py Outdated
return self.canonical_company.siret
raise ValidationError("Could not find authoritative SIAE from ASP source")

def has_job_descriptions_not_updated_for(self, days=60):

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.

nit: Ça n´aurais sans doute jamais trop servi coté métier mais tu aurais sans doute pu directement accepter un timedelta(), ce qui dans les tests t'aurais permis de passer timedelta(seconds=0) et donc de ne pas avoir besoin de freeze_time() :).

Comment thread tests/companies/test_models.py Outdated
Comment thread tests/companies/test_models.py Outdated
Comment thread itou/templates/layout/base.html Outdated
Comment thread itou/templates/layout/base.html Outdated
Comment thread itou/templates/layout/base.html Outdated
@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch from c325b6c to 0064cc2 Compare June 6, 2025 09:22
@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch from 0064cc2 to 5ce8d37 Compare June 6, 2025 09:27
@EwenKorr

EwenKorr commented Jun 6, 2025

Copy link
Copy Markdown
Contributor Author

Arf, c'est pas updated_at mais last_employer_update_at qu'il faut regarder 🤦

@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch from 5ce8d37 to f0c9784 Compare June 6, 2025 09:46
@EwenKorr EwenKorr added 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC and removed 1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC labels Jun 6, 2025
@github-actions

github-actions Bot commented Jun 6, 2025

Copy link
Copy Markdown

🥁 La recette jetable est prête ! 👉 Je veux tester cette PR !

@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch 3 times, most recently from e2cc3db to f097028 Compare June 6, 2025 10:12
@EwenKorr EwenKorr force-pushed the ewen/employer_banner branch from f097028 to 84c2b9f Compare June 6, 2025 10:13
@EwenKorr EwenKorr enabled auto-merge June 6, 2025 10:14
@EwenKorr EwenKorr changed the title Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées depuis 60 plus [GEN-2498] Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées depuis plus de 60 jours [GEN-2498] Jun 6, 2025
@EwenKorr EwenKorr changed the title Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées depuis plus de 60 jours [GEN-2498] Employeur : ajout d'une bannière leur rappelant la future dépublication des fiches de poste non actualisées [GEN-2498] Jun 6, 2025
@EwenKorr EwenKorr added this pull request to the merge queue Jun 6, 2025
Merged via the queue into master with commit 008524f Jun 6, 2025
14 checks passed
@EwenKorr EwenKorr deleted the ewen/employer_banner branch June 6, 2025 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1-recette-jetable [Payé à l’heure] Crée une recette jetable sur CC ajouté Ajouté dans le changelog.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants