Skip to content

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
Member

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. 🤷

"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
Member

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
Member

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.

"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
Member

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
Member

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 ;).

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

4 participants