feat: Ajout des champs pour mieux gerer les forms #57

Merged
ariasat merged 9 commits from log_forms into main 2026-07-19 20:38:25 -04:00
Contributor

Closes #52
Closes #53

Closes #52 Closes #53
feat(UI):Ajout boutton pour reinisialiser les filtres
All checks were successful
/ build (pull_request) Successful in 1m10s
/ help (pull_request) Successful in -6s
1be85b3fd2
Merge branch 'main' into log_forms
All checks were successful
/ build (pull_request) Successful in 1m10s
/ help (pull_request) Successful in -6s
c2cea6ee4d
@ -76,0 +82,4 @@
hx-patch="/admin/soumissions/{{ .ID }}"
hx-trigger="change"
hx-vals="js:{resolved: event.target.checked}"
hx-confirm="{{ if .Resolved }}Retirer le statut résolu de cette soumission ?{{ else }}Marquer cette soumission comme résolue ?{{ end }}"
Owner

mieux d'utilise un vrai modal au lieu d'utiliser des alerts

mieux d'utilise un vrai modal au lieu d'utiliser des alerts
dnova marked this conversation as resolved
@ -76,0 +85,4 @@
hx-confirm="{{ if .Resolved }}Retirer le statut résolu de cette soumission ?{{ else }}Marquer cette soumission comme résolue ?{{ end }}"
hx-include="#adminSoumissionsFilters"
hx-target="#soumissions-list-panel"
hx-swap="innerHTML">
Owner

aussi quand on annule le alert, le checkbox est checked, il devrait aussi avoir plus de confirmation que le form etait resolu/archive.

tant qu'on utilise des checkbox on pourrait aussi utiliser bulk archive comme on a fait avec les signalements dans cms

aussi quand on annule le alert, le checkbox est checked, il devrait aussi avoir plus de confirmation que le form etait resolu/archive. tant qu'on utilise des checkbox on pourrait aussi utiliser bulk archive comme on a fait avec les signalements dans cms
dnova marked this conversation as resolved
Owner

est-ce que je suis fou ou il y a genre 3 differents fonts dans ce page haha

est-ce que je suis fou ou il y a genre 3 differents fonts dans ce page haha
Author
Contributor

@ariasat wrote in #57 (comment):

est-ce que je suis fou ou il y a genre 3 differents fonts dans ce page haha

yep bro cest just poppins comme dabs

@ariasat wrote in https://git.agecem.com/ariasat/agecem-org/pulls/57#issuecomment-8686: > est-ce que je suis fou ou il y a genre 3 differents fonts dans ce page haha yep bro cest just poppins comme dabs
db.go Outdated
@ -227,0 +232,4 @@
return true, nil
}
func (c PostgresClient) BulkUpdateFormSubmissions(ctx context.Context, ids []int, resolved bool) (nUpdated int64, err error) {
Owner

ngl je pensais que resolve c'etait juste archive c'est pour ca que j'ai dit d'utiliser bulk haha, mais ca marche aussi. Si cette methode update seulement resolve, c'est mieux de juste l'appeler ResolveFormSubmissions. Meme chose pour les autres fonctions dans le workflow

ngl je pensais que resolve c'etait juste archive c'est pour ca que j'ai dit d'utiliser bulk haha, mais ca marche aussi. Si cette methode update seulement resolve, c'est mieux de juste l'appeler ResolveFormSubmissions. Meme chose pour les autres fonctions dans le workflow
Author
Contributor

yes theoriquement il marche comme un archive, cest just pour le soft delete, mais la jai fait que ca soit en bulk avant cetait un par un, mais oui pour le nom ca fait plus de sense.

yes theoriquement il marche comme un archive, cest just pour le soft delete, mais la jai fait que ca soit en bulk avant cetait un par un, mais oui pour le nom ca fait plus de sense.
dnova marked this conversation as resolved
refactor: changement de nom pour etre plus coherent par resolve form
All checks were successful
/ build (pull_request) Successful in 1m35s
/ help (pull_request) Successful in -7s
221f3f01c4
ariasat deleted branch log_forms 2026-07-19 20:38:25 -04:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
ariasat/agecem-org!57
No description provided.