Produits Entr'ouvert » Chrono

(yep, merci Fred, je ne suis vraiment pas inspiré pour le message)

Patch qui reprend https://styleguide.entrouvert.com/motifs/etat-initial-vide/

J'ai mis le texte en anglais mais je pensais à ça en français :
"Cet écran reprend les agendas que vous pouvez consulter.
Malheureusement, il n'y en a encore aucun de configuré pour qu'il vous soit accessible."

j'en verrais même peut-être une variation avec l'icône remplacée par un :(

• qui utilise le smiley fa-frown-o ("\f119") de la fonte awesome,
• et de dériver le style "infonotice" (pour pas inventer un nouveau truc) ?

qui utilise le smiley fa-frown-o ("\f119") de la fonte awesome,

Pour l'icône d'info on a une vraie image, on peut pour moi faire pareil ici et ça pourra être plus élégant; genre dans l'idée rapidement fait la capture attachée. (et oui côté classe ça serait un big-msg-sorry comme on a big-msg-nfo).

Est-ce qu'il faut que je revienne à la ligne entre les deux phrases ?
(je n'ai pas trouvé comment le faire proprement)

0001: au passage j'ai corrigé un test

0002:

J'ai fait le tour des urls de /manage/. Mis à part '/manage/agendas/events.csv' qui fourni publiquement un exemple du format des événements à importer, je n'ai rien vu qui ne soit pas contrôlé.
Pour le template 403 de base j'ai mis ce message :

Forbidden access, sorry.
Some configuration should be set if you need to access this page.

J'ai inclus le message discuté plus haut pour les utilisateurs qui arrivent sur la page d’accueil sans avoir accès à aucun calendrier, directement dans le template de la vue. Et pour renvoyer une 403, j'ai redéfini django/views/generic/list.py::BaseListView:get()

J'ai appliqué la même modification aux agendas d'indisponibilités.

• test_agenda_day_view et test_virtual_agenda_day_view parce que l'on fait 2 requêtes SQL de moins (j'ai pas chercher à comprendre).
• test_unavailability_calendar_homepage_permission qui utilise le bouton d'action présent sur la page d’accueil, qui à présent est retournée (à l'identique) en 403.

0003: Je ne sais pas si je dois ou non supprimer le code de manager_required (qui n'est plus utilisé) car je me dis que ça pourrait resservir.

0002 : message adapté sur la page d'accueil (remarques prises en compte)

Pour moi là on est déjà sur une page à l'intérieur, on peut aller sur l'erreur générique.

0003: corrige la régression sur /manage/unavailability-calendars/ afin de continuer
à retourner une 403 pour les utilisateurs qui n'ont pas de droits sur les agendas indisponibilité.

Je me dis que celui-là pourrait être distribué par Gadjo

0004 : template 403 qui inclut un message générique posé sur gadjo.
Parce qu'un template 403 chargé directement sur gadjo ne ferait pas apparaître le titre. (cf https://dev.entrouvert.org/issues/48367#note-3)

Le template gadjo ressemblerait à ça :

{% load i18n %}

<div class="big-msg-sorry">
  <p>{% trans 'Forbidden access, sorry.' %}</p>
  <p>{% trans 'Some configuration should be set if you need to access this page.' %}</p>
</div>

Yep. En fait j'ai l'impression qu'il faudra de toute façon poser un template 403 dans chrono pour pouvoir charger celui de gadjo.
Quoi qu'il en soit, je réalise que 0004 n'a pas sa place ici, je le retire.

Je serais pour réunir tout ça en un seul patch, corriger "didicated" en "dedicated" dans l'intitulé, et ça sera ok.

commit ef55e9de5680beccb37a2f31650d3fd39e0eeda0
Author: Nicolas ROCHE <nroche@entrouvert.com>
Date:   Thu Dec 3 10:54:56 2020 +0100

    manager: set a dedicated 403 response on home page (#48367)