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

Supprimer les signatures des PJ lors de la fusion #75

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vinriviere
Copy link

Je propose ce patch pour corriger l'issue #56. C'est en PROD à Paris 1 depuis quelques semaines, et cela nous donne satisfaction.
Je souhaite qu'il soit intégré (sous une forme ou une autre) à la version officielle d'eCandidat.

Point clé :
Depuis quelques mois, les relevés de notes de l'Université Paris 1 sont signés numériquement. Lorsque des candidats issus de Paris 1 vont candidater sur les applications eCandidat des autres établissements, ils vont déposer leurs relevés de notes dans les PJ. Comme ils sont signés, lors de la fusion avec les autres pages du dossier eCandidat, la signature va devenir invalide. Nous ne souhaitons pas que les dossiers des étudiants Paris 1 se retrouvent invalides dans les autres établissements.

Ce patch supprime les signatures des documents fusionnés. Ainsi, le dossier eCandidat fusionné ne contiendra jamais de bribes de signatures invalides. Si les établissements n'ont pas activé la signature numérique dans eCandidat, le dossier restera non signé, mais valide.

NB : Concernant la forme, je n'ai pas fait d'effort particulier, car je ne sais pas si ce patch va être mergé tel quel. Bien sûr, si besoin, je suis prêt à le raffiner avec des commentaires, etc. Mais peut-être que ce patch ne sera pas utilisé tel quel, et servira juste de source d'inspiration pour l'intégration officielle. Tout ce que je souhaite, c'est que cette fonctionnalité soit intégrée dans la version officielle, d'une manière ou d'une autre.

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.

1 participant