-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/config events #626
base: main
Are you sure you want to change the base?
Feat/config events #626
Conversation
a35ac10
to
100a8fd
Compare
8d52395
to
665db4c
Compare
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, ça fonctionne super bien !
Je te demande toutefois quelques renommages sur les routes pour les avoir en français comme sur le reste du site et une réorganisation de la page niveau de service pour mieux en hiérarchiser le contenu et avoir l'accès au RSS.
Le layout de la page Niveau de service est encore incomplet, j'ai marqué unresolved mon commentaire (qui était un peu long)
|
8032709
to
c0219b5
Compare
Corrige #618
Il manque encore:
integration texte riche pour le détailflux RSSon n'affiche pas le lien pour aller sur la page de configuration, mais il n'y a pas d'affichage de page 403 si on rentre manuellement l'URL et que l'on a pas les bons droits (pas accès à l'entrepôt ou pas le droit annexe) => nécessite un rework global pour résoudre ce pointAutres points:
cache: "no-store"
pour charger le fichiers des alertes et éviter d'utiliser le cache navigateur qui pose problème