-
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 : Ajouter des encarts pour les prochains webinaires #4592
Conversation
7a654c7
to
94070ce
Compare
@@ -130,6 +133,18 @@ def render_stats(request, context, params=None, template_name="stats/stats.html" | |||
|
|||
# Key value pairs in context override preexisting pairs in base_context. | |||
base_context.update(context) | |||
if "pilotage_webinar_banners" not in base_context: |
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.
Facilitation à surcharger 👍
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.
Merci pour avoir inclus le lien à #4207, il a fournis bien la contexte. C'est visible que ce fonctionnalité fait un bonne évolution 👍
base_context["pilotage_webinar_banners"] = [ | ||
{ | ||
"title": "Des questions sur l’utilisation de vos tableaux de bord ?", | ||
"description": "Nous y répondons lors d’un webinaire questions / réponses animé chaque mois. Inscrivez-vous à la prochaine session prévu jeudi 29 août de 14h à 14h45.", # noqa: E501 | ||
"url": "https://app.livestorm.co/itou/le-pilotage-de-linclusion-professionnels-de-liae-questions-reponses-sur-les-tableaux-de-bord-1?s=67c316de-9dc5-4ec7-a469-6698507fa312", # noqa: E501 | ||
"is_displayable": lambda: timezone.localdate() <= datetime.date(2024, 8, 29), | ||
} | ||
] | ||
base_context["pilotage_webinar_banners"] = [ | ||
banner for banner in base_context["pilotage_webinar_banners"] if banner["is_displayable"]() | ||
] |
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.
Il n'y a pas d'intérêt à gérer les détails des webinars avec le site admin via un modèle ? Ça me paraît un besoin récurrent
@@ -20,6 +20,29 @@ <h1>{{ page_title }}</h1> | |||
{% endif %} | |||
{% endblock %} | |||
|
|||
{% block title_messages %} | |||
{{ block.super }} | |||
{% for banner in pilotage_webinar_banners %} |
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.
Je vois que ce n'est plus utilisé dans itou/templates/dashboard/dashboard.html
, je pense qu'il n y a pas avantage de déplacer ce composant vers un {% include %}
👍
context = { | ||
"pilotage_webinar_banners": [ | ||
{ | ||
"title": "Des questions sur la prise en main de ce nouveau tableau de bord ?", | ||
"description": "Nous y répondons en direct lors d’un webinaire de questions / réponses, dédiés aux Conseils départementaux, organisés le mardi 17 septembre à 14h", # noqa: E501 | ||
"url": "https://app.livestorm.co/itou/le-pilotage-de-linclusion-webinaire-questions-and-reponses-pour-les-conseils-departementaux?s=f83e62e2-42a3-4cc1-9772-c963d2be2c00", # noqa: E501 | ||
"is_displayable": lambda: timezone.localdate() <= datetime.date(2024, 9, 17), | ||
} | ||
] | ||
} | ||
return render_stats_cd(request=request, page_title="Facilitation des embauches en IAE", extra_context=context) | ||
|
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.
Y-a-t'il des règles explicite pour la montage des divers webinars ? Il me semble que c'est par Institution ?
Je pense que au long terme ça va être difficile à lire / maintenir s'ils sont hardcodés par vue, et ils sont pas testés. Ça vaut peut-être un modèle Webinar
qui gère le contenu et peut-être même les règles d'affichage ?
🤔 Pourquoi ?
🍰 Comment ?
Réutilisation de ce qui avais été fait dans #4207, sauf que cette fois les bannières sont afficher sur les pages des TB.