-
Notifications
You must be signed in to change notification settings - Fork 24
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
Conversation
itou/www/dashboard/views.py
Outdated
"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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
timezone.localdate()
?
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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. 🤷
itou/www/dashboard/views.py
Outdated
@@ -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), |
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
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 ;).
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
itou/www/dashboard/views.py
Outdated
@@ -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), |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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 ;).
Si tu merges la PR demain, tu n’auras plus besoin de faire une liste pour les alertes pilotage, ni de comparer des dates 😇 😉 |
There was a problem hiding this 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 ;).
itou/www/dashboard/views.py
Outdated
@@ -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), |
There was a problem hiding this comment.
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.
041f333
to
c000809
Compare
🤔 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 ;).