Julien Sagot
Julien Sagot
IMHO: - This is a good proposal - `print_sosa` should go in another module (let say `sosaDisplay.ml`) in order to make it possible to put `SosaMain` in the right sublibrary....
À moins d'avoir une proposition de spécifications plus précise, je ferme l'issue.
> ET qu'en supprimant l’événement, on puisse supprimer aussi cette date plancher SI elle n'a pas été modifiée Je parlais plutôt de cette partie. Sinon effevtivement c'est très faisable. Mais...
Je réouvre. À mettre au clair mais la fonctionnalité semble pertinente :+1:
Seems good to me, but I need to double-check this an test few gedcoms before merging.
I would prefer a generic `Unauthorized` or `Not_found`. `Unauthorized` seems to be the current behavior. The message could be included in the `401` response body.
Oui bien entendu, comme je le disais, on peut inclure le message dans le body de la `401` qqch comme ```ocaml let content = (* contruction de la page *)...
PS : le close n'était pas voulu, mais il faudrait wrapper ta modif dans le output error. Pour ce qui est du ``` | None -> include_template conf [] "index"...
Tout à fait, mais si tu veux utiliser `SrcfileDisplay.print_start conf base` c'est faisable. Je te montre comment dès que j'ai deux minutes (cf les plugins cgl et no_index qui "print"...
Je préfère une solution qui permet de modifier la comportement via GWPARAM. Ce n'est pas du tout évident qu'on veuille utiliser un template pour cette page. Surtout que je compte...