Skip to content

Permet de réordonner les pages de l'échantillon#613

Merged
bellangerq merged 20 commits intomainfrom
215-changer-lordre-des-pages-depuis-les-paramètres-de-laudit
Jan 25, 2024

Hidden character warning

The head ref may contain hidden characters: "215-changer-lordre-des-pages-depuis-les-param\u00e8tres-de-laudit"
Merged

Permet de réordonner les pages de l'échantillon#613
bellangerq merged 20 commits intomainfrom
215-changer-lordre-des-pages-depuis-les-paramètres-de-laudit

Conversation

@bellangerq
Copy link
Copy Markdown
Collaborator

@bellangerq bellangerq commented Jan 17, 2024

To do:

  • Corriger le bug d'ordre des pages (audit & rapport)

closes #215

@netlify
Copy link
Copy Markdown

netlify Bot commented Jan 17, 2024

Deploy Preview for confiture ready!

Name Link
🔨 Latest commit c29dd60
🔍 Latest deploy log https://app.netlify.com/sites/confiture/deploys/65b290f701c83b0008631cf8
😎 Deploy Preview https://deploy-preview-613--confiture.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@bellangerq bellangerq marked this pull request as ready for review January 18, 2024 10:50
@bellangerq bellangerq force-pushed the 215-changer-lordre-des-pages-depuis-les-paramètres-de-laudit branch from ac40a18 to 47f4339 Compare January 18, 2024 14:59
Comment thread confiture-web-app/src/components/report/ReportErrors.vue Outdated
Copy link
Copy Markdown
Collaborator

@hissalht hissalht left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Je pense qu'il faudrait ajouter une information en base pour l'ordre des pages. Parce que là le serveur renvoie les pages dans l'ordre dans lesquelles elles sont créées en base. Donc, si on essaye de changer l'ordre après avoir créé l'audit, ça marchera pas.

Je pense qu'on peut juste ajouter une propriété order dans la table AuditPage. Et quand on doit afficher une liste de page (rapport, page des paramètres, onglets, etc), on fait juste orderBy(pages, 'order')

Comment thread confiture-web-app/src/components/audit/AuditGeneralInformationsForm.vue Outdated
@bellangerq bellangerq force-pushed the 215-changer-lordre-des-pages-depuis-les-paramètres-de-laudit branch from 47f4339 to 0e288c7 Compare January 19, 2024 12:00
Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
Comment thread confiture-rest-api/src/audits/audit.service.ts Outdated
Comment thread confiture-rest-api/src/audits/dto/create-audit.dto.ts Outdated
@benoitdequick
Copy link
Copy Markdown
Contributor

C'est top 🎉
J'ai quand même trouvé 2-3 petits trucs 😅

  • Lors de la modification des paramètres de l'échantillon, la confirmation du changement de position de page n'est pas vocalisé.
  • Si j'ajoute une page à l'échantillon, quelque soit la position que j'indique, elle est placée sur le dernier onglet (mais je peux la modifier ensuite)

@bellangerq bellangerq requested a review from hissalht January 24, 2024 13:47
@benoitdequick
Copy link
Copy Markdown
Contributor

j'ai toujours le bout de code vocalisé une fois que j'ai validé les paramètres :(

@bellangerq
Copy link
Copy Markdown
Collaborator Author

bellangerq commented Jan 25, 2024

j'ai toujours le bout de code vocalisé une fois que j'ai validé les paramètres :(

Yes, j'ai fait un ticket (#627) là-dessus, c'était déjà le cas avant mais ça n'est pas lié à cette feature.

@bellangerq bellangerq force-pushed the 215-changer-lordre-des-pages-depuis-les-paramètres-de-laudit branch from 5689a55 to c29dd60 Compare January 25, 2024 16:48
@bellangerq bellangerq merged commit 049c8b0 into main Jan 25, 2024
@bellangerq bellangerq deleted the 215-changer-lordre-des-pages-depuis-les-paramètres-de-laudit branch January 25, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Changer l'ordre des pages depuis les paramètres de l'audit

3 participants