-
Notifications
You must be signed in to change notification settings - Fork 10
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
RI-192: [admin/front] permet d'archiver plusieurs offres depuis le listing d'offres #203
Conversation
mathusha-sdv
commented
Nov 27, 2021
•
edited
Loading
edited
- bug au niveau de la sélection des offres, quand je sélectionne deux offres et j'archive, ça fonctionne puis quand j'en sélectionne une et j'archive, il prend les deux sélections et les archive les deux --> résolu, l'element etait encore en selection
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 serais dispo dans l'après midi pour en discuter parce qu'en regardant mieux ce composant, je me dis qu'il y a peut être moyen de faire mieux ... Il faudait également voir avec @dylanbrudey pour harmoniser les designs des tableaux ... sans retour de sa part on est bloqué !
frontend-implicaction/src/app/core/services/api-endpoints.service.ts
Outdated
Show resolved
Hide resolved
frontend-implicaction/src/app/admin/jobs/components/jobs-table/jobs-table.component.ts
Outdated
Show resolved
Hide resolved
frontend-implicaction/src/app/admin/jobs/admin-jobs.component.ts
Outdated
Show resolved
Hide resolved
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.
Pense à vérifier que l'archivage d'un job unique fonctionne également 😉
frontend-implicaction/src/app/admin/jobs/admin-jobs.component.ts
Outdated
Show resolved
Hide resolved
frontend-implicaction/src/app/admin/jobs/admin-jobs.component.ts
Outdated
Show resolved
Hide resolved
frontend-implicaction/src/app/admin/jobs/admin-jobs.component.ts
Outdated
Show resolved
Hide resolved
frontend-implicaction/src/app/admin/jobs/admin-jobs.component.ts
Outdated
Show resolved
Hide resolved
frontend-implicaction/src/app/admin/jobs/components/jobs-table/jobs-table.component.ts
Outdated
Show resolved
Hide resolved
…sting d'offres * bug au niveau de la selection des offres, quand je selectionne deux ofres et j'archive, ça fonctionne puis quand j'en séléctionne une et j'archive, il prend les deux séléctions et les archive les deux
…sting d'offres * bug au niveau de la selection des offres, quand je selectionne deux ofres et j'archive, ça fonctionne puis quand j'en séléctionne une et j'archive, il prend les deux séléctions et les archive les deux
…sting d'offres * code review
…sting d'offres * code review * suppression composant jobs-table
* clean fichiers obsolètes
* clean fichiers obsolètes
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.
une toute petit correction et c'est tout bon 👍
frontend-implicaction/src/app/admin/jobs/admin-jobs.component.html
Outdated
Show resolved
Hide resolved
…html Co-authored-by: matthieuaudemard <matthieu.audemard@gmail.com>
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.
Si après avoir testé tu estimes que ça fonctionne bien tu peux merger 😉