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

Open
dnova wants to merge 8 commits from log_forms into main
Collaborator

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
Collaborator

@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
All checks were successful
/ build (pull_request) Successful in 1m34s
/ help (pull_request) Successful in -7s
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin log_forms:log_forms
git switch log_forms

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff log_forms
git switch log_forms
git rebase main
git switch main
git merge --ff-only log_forms
git switch log_forms
git rebase main
git switch main
git merge --no-ff log_forms
git switch main
git merge --squash log_forms
git switch main
git merge --ff-only log_forms
git switch main
git merge log_forms
git push origin main
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.