-
Notifications
You must be signed in to change notification settings - Fork 4
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
News list improvements #1009
base: taiste
Are you sure you want to change the base?
News list improvements #1009
Conversation
976cfad
to
96f5874
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.
Reste plus qu'à mettre un warning quand on modère un événement répété et ça me parait bon
@@ -0,0 +1,39 @@ | |||
import { exportToHtml } from "#core:utils/globals"; | |||
import Alpine from "alpinejs"; |
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.
Tu veux pas importer Alpine ici parce que c'est déjà importé en global.
Tu peux juste utiliser Alpine comme ça directement, c'est dans les globals du typescript avec les bons types.
Regarde easymde-index.ts pour un exemple
newsId: newsId as number, | ||
loading: false, | ||
|
||
async moderateNews() { |
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.
Ce serait bien de pouvoir forcer le reload du calendrier quand on fait ça pour avoir une expérience plus smooth.
Tu peux mettre un x-ref dessus et utiliser refetchEvents https://fullcalendar.io/docs/Calendar-refetchEvents
@@ -39,14 +39,6 @@ | |||
<a rel="nofollow" target="#" class="share_button twitter" href="https://twitter.com/intent/tweet?text={{ news.get_full_url() }}">{% trans %}Tweet{% endtrans %}</a> | |||
{%- endmacro %} | |||
|
|||
{% macro fb_quick(news) -%} |
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.
On peut les garder pour l'instant ? Je trouve que ça habille beaucoup la page d'acceuil, là ça fait vide :/
On mettra ça à jour dans une autre PR où on remplace ça par plus « moderne » genre instagram
Pour commencer, on résoud des problèmes de N+1 queries sur l'affichage des nouvelles sur la page d'accueil.
On change également le choix des nouvelles affichées :
On différencie les nouvelles modérées des nouvelles modérées avec un encart qui dit que la nouvelle est en attente de modération :
Si l'utilisateur connecté à la permission de modérer les nouvelles, l'alerte contient également un bouton pour modérer et un bouton pour supprimer. Le flux de modération/suppression des nouvelles :
Kooha-2025-01-20-17-38-01.webm
Après ça, les admins com auront plus d'excuse pour ne pas modérer les nouvelles.