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 : Ajouter des encarts pour les prochains webinaires #4592

Merged
merged 1 commit into from
Aug 28, 2024

Conversation

@rsebille rsebille self-assigned this Aug 22, 2024
@rsebille rsebille added the ajouté Ajouté dans le changelog. label Aug 22, 2024
@rsebille rsebille force-pushed the rsebille/c2/webinar-banners branch from 7a654c7 to 94070ce Compare August 27, 2024 17:17
@@ -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:
Copy link
Contributor

Choose a reason for hiding this comment

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

Facilitation à surcharger 👍

Copy link
Contributor

@calummackervoy calummackervoy left a 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 👍

Comment on lines +137 to +147
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"]()
]
Copy link
Contributor

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 %}
Copy link
Contributor

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 %} 👍

Comment on lines +362 to 373
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)

Copy link
Contributor

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 ?

@rsebille rsebille added this pull request to the merge queue Aug 28, 2024
Merged via the queue into master with commit 8854434 Aug 28, 2024
11 checks passed
@rsebille rsebille deleted the rsebille/c2/webinar-banners branch August 28, 2024 14:39
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.

2 participants