Skip to content
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

Problème export et preuve et mesure associé #439

Closed
protocolpaladin opened this issue May 16, 2024 · 10 comments · Fixed by #474 or #480
Closed

Problème export et preuve et mesure associé #439

protocolpaladin opened this issue May 16, 2024 · 10 comments · Fixed by #474 or #480
Assignees
Labels
bug Something isn't working

Comments

@protocolpaladin
Copy link
Contributor

*Describe the bug
L’ajout d’une mesure appliquée associé à une évaluation d’audit n’intègre pas la preuve automatiquement associé dans la partie preuve et aussi l’export du rapport exporte autant de pièce jointe pour une même mesure appliqué avec preuve ou preuve ajoute identique ajoutée manuellement. Et si plusieurs évaluations audits sont justifier par une même mesure ou preuve lors de l’export ces fichiers sont exportés plusieurs fois et crée un export conséquent en taille alors qu’on utilise le ma même mesure ou preuve plusieurs fois exemple pssi charte.

To Reproduce
Steps to reproduce the behavior:
Faire plusieurs évaluations d’audit d’un framework et associé une à plusieurs mesure similaire déjà associé à des preuves répéter pour chaque évaluation et ajouter les preuves identiques manuellement.

et exporter

Expected behavior
A clear and concise description of what you expected to happen.

Lors de l’ajout d’une mesure avec preuve associé la reporter automatiquement dans la partie preuve.

lors de l’utilisation d’une même mesure ou preuve pour une même évaluation ou plusieurs évaluation n’exporter qu’en seul fois le fichier dans l’export.

@ab-smith ab-smith added the bug Something isn't working label May 16, 2024
@Mohamed-Hacene
Copy link
Collaborator

Mohamed-Hacene commented May 17, 2024

Bonjour @protocolpaladin, merci pour ce retour.

Premièrement, il est normal que la preuve directement associée à la mesure ne soit pas renseignée dans la partie preuve de l'exigence. Cela permet permet d'éviter la duplication des preuves au sein de l'application car il existe deux moyens de lier une preuve à une exigence:

  • depuis une mesure appliquée
  • directement dans l'onglet preuve

Cela permet d'avoir des preuves exclusives à l'exigence qui peuvent être ajoutées en supplément des preuves des mesures appliquées, sans avoir besoin de créer une mesure juste pour cela.

Cependant, il n'est effectivement pas normal que ces preuves soient dupliquées dans le zip d'export, et nous vous remercions de nous l'avoir indiqué, une correction devrait arriver dans la journée.

@Mohamed-Hacene Mohamed-Hacene self-assigned this May 17, 2024
@protocolpaladin
Copy link
Contributor Author

bonjour @Mohamed-Hacene merci pour le retour.

suggestion :
Cela induit en erreur le fait qu’une preuve associé à une mesure ne soit pas tag dans la partie preuve nativement je trouve d’où la remontée et je pense que tag automatiquement la preuve ou afficher clairement dans l’application les mesure avec preuve sont automatiquement incluses et qu’il n’y a pas besoin de les rajouter empêcherai de se poser la question et d’autre ticket à venir peut être.

@Mohamed-Hacene
Copy link
Collaborator

Mohamed-Hacene commented May 18, 2024

Effectivement, je comprends mieux votre remarque. Je prends note de cela et nous verrons comment améliorer l'ergonomie/compréhension des preuves associées à une exigence.

Une fois le problème de duplication des preuves dans le rapport réglé, je laisserai cette issue ouverte pour garder une trace de cette demande et que vous soyez notifié lorsque nous travaillerons dessus, pour éventuellement avoir un avis externe.

Merci beaucoup pour votre suggestion !

@Mohamed-Hacene
Copy link
Collaborator

Bonjour @protocolpaladin,

Le problème de duplication des preuves a l'air d'être réglé sur la branche main, pourriez-vous essayer de votre côté sur la dernière version de la branche main afin d'être sûr que tout est ok de votre côté également ?

J'ai aussi ouvert la PR #474 afin d'ajouter des indications sur les différentes manières de lier des preuves à une exigence.

@Mohamed-Hacene Mohamed-Hacene linked a pull request May 22, 2024 that will close this issue
@protocolpaladin
Copy link
Contributor Author

Toujours le même problème d’après mes tests (j’ai build localement la version main actuel du git)

2 évaluations dans un même audit avec mesure et preuve similaire et 4 fois la même PJ dans l’export zip

@Mohamed-Hacene
Copy link
Collaborator

D'accord, c'est étrange je n'arrive plus à reproduire ce comportement pourtant je sais que ça m'est également arrivé juste avant que vous reportiez cette issue.

Pouvez-vous me préciser quel framework et quel type de preuve vous avez utilisé pour que je reproduise à l'identique votre use case ?

@protocolpaladin
Copy link
Contributor Author

protocolpaladin commented May 22, 2024

Norme iso version française avec .docx mais pareil également avec pdf ou image sur les premiers tests à l’ouverture de ce PR.

@Mohamed-Hacene Mohamed-Hacene linked a pull request May 23, 2024 that will close this issue
@Mohamed-Hacene
Copy link
Collaborator

Mohamed-Hacene commented May 23, 2024

Ok @protocolpaladin, I did not reproduce the error (because it probably depends on the environment) but I think I found an inconsistency that should be the cause of this issue in your case.

Could you try to export your audit on the last PR #480 I linked ?

@protocolpaladin
Copy link
Contributor Author

Toujours le même problème d’après mes tests en rebuild le git main dans sa version actuelle

@Mohamed-Hacene
Copy link
Collaborator

Sur main c'est normal, la branche n'a pas été poussée encore. Si vous n'avez pas la capacité de tester sur la branche, je reviendrai vers vous pour que vous testiez quand nous aurons mis main à jour.

Encore merci pour ce feedback !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants