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

[PatientEditModal] : Implement Patient Edit Form #267

Open
6 of 7 tasks
remixxx31 opened this issue Jul 11, 2024 · 12 comments
Open
6 of 7 tasks

[PatientEditModal] : Implement Patient Edit Form #267

remixxx31 opened this issue Jul 11, 2024 · 12 comments
Assignees

Comments

@remixxx31
Copy link
Contributor

remixxx31 commented Jul 11, 2024

[PatientEditModal]: Implement Patient Edit Form

Checklist

  • Create a PatientEditModal component.
  • Include the following fields in the form:
    • Patient ID (readonly)
    • Patient Name
    • Patient Birth Date
    • Patient Sex
  • Add a Save button to submit the form.
  • Add a Cancel button to close the modal without saving.
  • Implement form submission logic.
  • Style the modal usi
    Screenshot from 2024-07-12 00-10-54

Questions

  1. What is the API endpoint for updating patient details?
  1. Should there be any additional fields or buttons?
@remixxx31 remixxx31 self-assigned this Jul 11, 2024
@remixxx31 remixxx31 changed the title [Component] PatientEditModal: Implement Patient Edit Form [PatientEditModal] : Implement Patient Edit Form Jul 11, 2024
@remixxx31
Copy link
Contributor Author

J'ai trouvé cette API patients/id/modify, je pense que c'est la bonne ?
Après je dois appeller l'api jobs dans la foulée avec l'ID de la response de l'APi précèdente exemple:https://otjs.ddns.net/api/jobs/0b2db275-3492-444b-8cc9-a4cc27368c4f?
Ou juste invalider la clé jobs ?
On peut supprimmer n'importe quel champ de type PatientMainDIcomTags à part ID ?

Voici le type de Payload:
{
"Replace": {
"PatientID": "\tef28989e-c4bf-4360-8242-e0287ca59fb7"
},
"Remove": [
"PatientSex"
],
"RemovePrivateTags": true,
"Force": true,
"Synchronous": false,
"KeepSource": false
}

Il y a un truc que je comprends pas , c'est qu'on ait obligé de spécifier le patient ID qui ne doit pas être vide , ni le même qu'avant, par contre si on copie-colle l'ancien ID ça fonctionne quand-même ?

@salimkanoun

@salimkanoun
Copy link
Contributor

Yes c'est la bonne api
Oui faut appeler job avec l id, mais réutilise le composant de suivi de job que t a fait pour le retrieve, la logique est la même tu a deja implémenté tout ca

@salimkanoun
Copy link
Contributor

Non y a des tag dicom qui ne peuvent pas être supprimés, je pense qu'on a géré ça dans Orthanc tools js tu dois pouvoir les retrouver dans le code (ça dépend du niveau d édition patient / study ou séries)

@salimkanoun
Copy link
Contributor

RemovePrivateTags": true, => mettre false
"KeepSource": false => ce boolean dépend de l'interface graphique tu met une case à cocher pour savoir si tu keep le dicom original ou pas

@remixxx31
Copy link
Contributor Author

Coucou Salim je t'ai envoyé un message dans Discord.
J'ai un soucis avec l'API de Modify Patient, je ne comprends vraiment pas ce qui me manque!
https://discord.com/channels/@me/1059479477870141481/1263494694357438545

@salimkanoun

@remixxx31
Copy link
Contributor Author

Est-ce que ce type de formulaire va bien pour l'edition de Patient ?
Je n'ai pas mis de newValue j'ai mis directement la valeur à editer ça va ou pas ?

Image

@salimkanoun

@remixxx31
Copy link
Contributor Author

Est-ce que ce type de formulaire va bien pour l'edition de Patient ? Je n'ai pas mis de newValue j'ai mis directement la valeur à editer ça va ou pas ?

Image

@salimkanoun

Est-ce qu'il faut que je retire les champs si ils ont nulls?

@remixxx31
Copy link
Contributor Author

Est-ce que ce type de formulaire va bien pour l'edition de Patient ? Je n'ai pas mis de newValue j'ai mis directement la valeur à editer ça va ou pas ?
Image
@salimkanoun

Est-ce qu'il faut que je retire les champs si ils ont nulls?

La valeur PatienId ne doit pas être null , on est d'accord si une quelconque modification est soumis par le formulaire le patientId doit être différent ?

@salimkanoun
Copy link
Contributor

Non le patient id peut rester le même aussi, il doit avoir une valeur mais test pas obligé de la changer.

Le reste me semble bon, tu a des composants similaires a faire pour l'édition des studies et series, c'est les items qui changent

@salimkanoun
Copy link
Contributor

Les champ delete tu les grise comme ça c'est réversible

@remixxx31
Copy link
Contributor Author

Les champ delete tu les grise comme ça c'est réversible

Quand je clique sur la poubelle tu veux que l'input devienne disable , c'est ça ?

@salimkanoun
Copy link
Contributor

Les champ delete tu les grise comme ça c'est réversible

Quand je clique sur la poubelle tu veux que l'input devienne disable , c'est ça ?

yes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants