Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pilotage : Mettre un encart pour promouvoir un webinaire pour les SIAE #4207

Merged
merged 3 commits into from
Jun 12, 2024

Conversation

rsebille
Copy link
Contributor

🤔 Pourquoi ?

https://www.notion.so/gip-inclusion/Mettre-un-encart-pour-promouvoir-webinaire-sur-les-emplois-dans-la-page-stats-du-pilotage-pour-les-S-800a904071904de1bd68711b312b320e?pvs=4

🍰 Comment ?

Léger réusinage au passage afin de gérer les dates d'affichage sans avoir besoin de 📆 et de 🧠 les jours J.
Je nettoyais les deux (bientôt) expirés la semaine prochaine ;).

@rsebille rsebille added the ajouté Ajouté dans le changelog. label Jun 11, 2024
@rsebille rsebille self-assigned this Jun 11, 2024
"description": "Des difficultés de recrutement ? Le mardi 9 juillet à 14h, le Pilotage de l’inclusion organise un webinaire pour vous aider à vous saisir des données utiles à la réalisation d’un bilan avec vos prescripteurs.", # noqa: E501
"url": "https://app.livestorm.co/itou/le-pilotage-de-linclusion-siae-difficultes-de-recrutement-faites-le-point-a-mi-annee-sur-vos-candidatures-and-prescriptions", # noqa: E501
"is_displayable": lambda: (
datetime.date(2024, 6, 17) <= timezone.now().date() <= datetime.date(2024, 7, 9)
Copy link
Contributor

Choose a reason for hiding this comment

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

timezone.localdate() ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dans mon esprit les local{date,time} sont plutôt pour convertir dans une TZ qu'avoir le now() mais si on se dit ça alors peut-être unifier l'utilisation partout ailleurs ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Par défaut ils appellent now, donc on s’en sert pas mal pour avoir la date courante. 🤷

@@ -167,6 +169,7 @@ def dashboard(request, template_name="dashboard/dashboard.html"):
"title": "Inscrivez-vous à un webinaire pour découvrir votre tout nouveau tableau de bord !", # noqa: E501
"description": "En juin, deux sessions vous sont proposées pour vous familiariser avec votre nouvel outil de suivi et d'analyse des résultats de vos prescriptions.", # noqa: E501
"url": "https://app.livestorm.co/itou/le-pilotage-de-linclusion-professionnels-missions-locales-et-cap-emploi-decouvrez-votre-nouveau-tableau-de-bord-personnalise-et-faites-le-point-sur-vos-prescriptions?type=detailed", # noqa: E501
"is_displayable": lambda: timezone.now().date() <= datetime.date(2024, 6, 11),
Copy link
Contributor

Choose a reason for hiding this comment

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

C’est normal que cette date soit passée ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Aujourd'hui oui :), c'est pour ça que j'indique :

Je nettoyais les deux (bientôt) expirés la semaine prochaine ;).

Copy link
Contributor

Choose a reason for hiding this comment

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

Bah, c’est qu’elle pourrait être supprimée dès maintenant.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

T'façon faudra supprimer l'autre bientôt, et il y a plus de chance que je le fasse si il en y a 2 plutôt que 1 ;).
Et c'est aussi la raison du is_displayable, ne pas courir après les bannières en fonction de quand c'est fait ou fusionnée.

@@ -180,6 +183,7 @@ def dashboard(request, template_name="dashboard/dashboard.html"):
"title": "Inscrivez-vous à un webinaire pour découvrir votre tout nouveau tableau de bord !", # noqa: E501
"description": "En juin, deux sessions vous sont proposées pour vous familiariser avec votre nouvel outil de suivi et d'analyse des résultats de vos prescriptions.", # noqa: E501
"url": "https://app.livestorm.co/itou/le-pilotage-de-linclusion-prescripteurs-de-laccueil-de-lhebergement-et-de-linsertion-decouvrez-votre-nouveau-tableau-de-bord-personnalise-et-faites-le-point-sur-vos-prescriptions?type=detailed", # noqa: E501
"is_displayable": lambda: timezone.now().date() <= datetime.date(2024, 6, 13),
Copy link
Contributor

Choose a reason for hiding this comment

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

Cette alerte ne sera bientôt plus visible.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oui :).

Je nettoyais les deux (bientôt) expirés la semaine prochaine ;).

@francoisfreitag
Copy link
Contributor

Si tu merges la PR demain, tu n’auras plus besoin de faire une liste pour les alertes pilotage, ni de comparer des dates 😇 😉

Copy link
Contributor Author

@rsebille rsebille left a comment

Choose a reason for hiding this comment

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

Si tu merges la PR demain, tu n’auras plus besoin de faire une liste pour les alertes pilotage, ni de comparer des dates 😇 😉

Les dates sont inclusives donc celle du 13 est encore nécessaire demain, et il y aura toujours besoin des dates pour celle commençant la semaine prochaine et se finissant pendant mes congés.
Et j'ai un très fort sentiment que les demandes de ce type ne vont pas disparaître donc j'anticipe ;).

@@ -167,6 +169,7 @@ def dashboard(request, template_name="dashboard/dashboard.html"):
"title": "Inscrivez-vous à un webinaire pour découvrir votre tout nouveau tableau de bord !", # noqa: E501
"description": "En juin, deux sessions vous sont proposées pour vous familiariser avec votre nouvel outil de suivi et d'analyse des résultats de vos prescriptions.", # noqa: E501
"url": "https://app.livestorm.co/itou/le-pilotage-de-linclusion-professionnels-missions-locales-et-cap-emploi-decouvrez-votre-nouveau-tableau-de-bord-personnalise-et-faites-le-point-sur-vos-prescriptions?type=detailed", # noqa: E501
"is_displayable": lambda: timezone.now().date() <= datetime.date(2024, 6, 11),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

T'façon faudra supprimer l'autre bientôt, et il y a plus de chance que je le fasse si il en y a 2 plutôt que 1 ;).
Et c'est aussi la raison du is_displayable, ne pas courir après les bannières en fonction de quand c'est fait ou fusionnée.

@rsebille rsebille force-pushed the rsebille/c2/siae-webinar-alert branch from 041f333 to c000809 Compare June 12, 2024 16:31
@rsebille rsebille added this pull request to the merge queue Jun 12, 2024
Merged via the queue into master with commit b8666e2 Jun 12, 2024
11 checks passed
@rsebille rsebille deleted the rsebille/c2/siae-webinar-alert branch June 12, 2024 16: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