Improve updating of patients #444

tomaspavlin · 2021-02-12T16:21:34Z

Kromě vylepšení updatu jsem opravil nějaké bugy, na které jsem narazil u přidávání pacientů, a předělal nějaké věci na FE, se kterými jsme dost válčili

BE

přidání dalších parametrů do updated api na BE + test
u přidávání pacientů se nenastavovalo previous_transplants, to jsem fixnul
swagger modely pro enumy

FE

přidání dalších parametrů do settings komponenty, kterou udělala @misslecter a využití této komponenty v nastavení pacientů
pro countries a blood groups teď používáme všude místo stringů enumy a další věci s enumy. Možná Closes BE -> FE enums conversion #465 @misslecter ?
vytvoření parserů interní modely->vygenerované swagger modely. Používám je při updatování a použil jsem je i u vytváření pacientů

Known bugs

po updatu donora bez recipienta se rozbije menu item na FE. Ale jinak změna proběhne v pořádku, po refreshnutí vše ok
update antigenů recipienta se nepropíše do transplant tabulky. Po refreshi zas vše ok. U donora to funguje

Issue na bugy #469
Swagger test bug issue mild-blue/swagger-unittest#5
BE enum issue #477

Closes #418

…ents # Conflicts: # txmatching/web/frontend/src/app/services/patient/patient.service.ts

…entu' into 418_improve_updating_of_patients # Conflicts: # txmatching/web/frontend/src/app/services/patient/patient.service.ts

- a lot of TODOOS to be fixed

…entu' into 418_improve_updating_of_patients # Conflicts: # txmatching/web/frontend/src/app/model/DonorEditable.ts # txmatching/web/frontend/src/app/model/PatientEditable.ts # txmatching/web/frontend/src/app/model/RecipientEditable.ts # txmatching/web/frontend/src/app/parsers/patient.parsers.ts # txmatching/web/frontend/src/app/services/patient/patient.service.ts

…entu' into 418_improve_updating_of_patients

…g_of_patients

…ing and fix creating patients previous_transplants

kubantjan · 2021-02-19T08:30:18Z

@tomaspavlin proc mas tady cancelled check? Prochazeji testy nebo ne?

kubantjan

Proklikano, projde mi to fajn, nezda se mi ze by ted neco krome tech veci na ktery uz je issue haprovalo :)

akorat u modify antibodies ted ukazujes code a vubec se neukazuje raw code on hover (kdezto u hla kodu to tak funguje) ale do budoucna se to bude v PR #462 stejne bude resit tak, ze tady se ukazou fakt jen ty raw kody, takze to klidne nechej takto, je to good.

jeste asi nasta a komenty na fe kod, ale backend kod je taky ok.

tomaspavlin · 2021-02-19T09:36:41Z

Proklikano, projde mi to fajn, nezda se mi ze by ted neco krome tech veci na ktery uz je issue haprovalo :)

akorat u modify antibodies ted ukazujes code a vubec se neukazuje raw code on hover (kdezto u hla kodu to tak funguje) ale do budoucna se to bude v PR #462 stejne bude resit tak, ze tady se ukazou fakt jen ty raw kody, takze to klidne nechej takto, je to good.

jeste asi nasta a komenty na fe kod, ale backend kod je taky ok.

V nastavení se antigeny i antibodies zobrazují jen jako raw kódy, naparsované kódy v nastavení nejsou vidět. Co mohu udělat je smazat matoucí popup u antigenů.

Na testy kouknu.

kubantjan · 2021-02-19T10:34:06Z

@tomaspavlin jop ten popup bych v tom pripade smazal

tomaspavlin · 2021-02-19T14:54:37Z

Jak už jsem řešil s @kubantjan , v rámci tohoto PR jsem zjistil, že některé starší unittesty byly špatně indentované a neprováděly se. Doteď pipelines toto neodhalili, ale nyní začaly padat. Opravil jsem tedy indentaci a obsah testů, aby s po novějších změnách kódu procházely.

Dále v tomto PR používám dědičnost v generování swagger modelů pro update donora a recipienta - to se nám kromě hezčího BE hodí na FE. Jenže swagger unittest na tom nefunguje dobře. Vytvořil jsem na to issue mild-blue/swagger-unittest#5 (@kubantjan @tomaskourim je v pořádku psát takové informace do veřejného repa?)

Dál nefunguje správně konverze enumů na BE, takže jsem kvůli tomu zatím upravil testy a vytvářím na to issue #477. Myslím, že to není způsobené tímto PR ale že už to tam bylo předtím. Je možné, že krevní skupiny kvůli tomu nefungují správně.

…g_of_patients

kubantjan

Vypada dobre, issue v swagger unittest mi prijde takto v pohode. Vesele bych mergoval (teda mozna by to mela cele jeste zchecknout nasta?)

misslecter

Super, jen pár komentů, a na všechna ta todočka bych vytvořila issue, ať se nám neztratí 🙂

txmatching/web/frontend/src/app/components/add-new-patient/add-new-patient.component.html

txmatching/web/frontend/src/app/components/patient-settings/sex/sex.component.html

txmatching/web/frontend/src/app/model/PatientEditable.ts

txmatching/web/frontend/src/app/model/enums/Sex.ts

…g_of_patients # Conflicts: # txmatching/web/frontend/src/app/components/patient-donor-detail/patient-donor-detail.component.scss # txmatching/web/frontend/src/app/components/patient-donor-detail/patient-donor-detail.component.ts # txmatching/web/frontend/src/app/services/patient/patient.service.ts

make pull request

6b8bca4

tomaspavlin added 20 commits February 12, 2021 19:12

Improve donor and recipient update api

38b4b18

Finish update api and introduce tests

53c55f1

Generate files

acbdd4d

Improve patient update service in FE to support everything

796e96b

Merge remote-tracking branch master into 418_improve_updating_of_pati…

52fab6d

…ents # Conflicts: # txmatching/web/frontend/src/app/services/patient/patient.service.ts

Merge remote-tracking branch 'origin/276_pekne_manualni_vkladani_paci…

82287a7

…entu' into 418_improve_updating_of_patients # Conflicts: # txmatching/web/frontend/src/app/services/patient/patient.service.ts

Use new component and create to-generated parsers

24e39c4

- a lot of TODOOS to be fixed

Fix some stuff

02dc5c2

Move active switch and recipient requirements to settings components

bab3435

Merge remote-tracking branch 'origin/276_pekne_manualni_vkladani_paci…

40f5087

…entu' into 418_improve_updating_of_patients

Comment

ba5920b

Merge remote-tracking branch 'origin/master' into 418_improve_updatin…

0594b6e

…g_of_patients

Solve some technical depts to make it working

3ac0002

Introduce editing waitingSince and previousTransplants, fix date pars…

161f981

…ing and fix creating patients previous_transplants

Show only needed controls

578b2bd

Improve enums in BE

f068f22

Improve enums in BE 2

21087d2

Solve TODOOs

3eacfee

Fix lint

16a6be3

tomaspavlin self-assigned this Feb 18, 2021

tomaspavlin requested review from misslecter, kubantjan and tomaskourim February 18, 2021 18:16

tomaspavlin marked this pull request as ready for review February 18, 2021 18:16

tomaspavlin mentioned this pull request Feb 18, 2021

Improve behaviour after patients update #469

Closed

kubantjan previously approved these changes Feb 19, 2021

View reviewed changes

tomaspavlin added 4 commits February 19, 2021 13:20

Remove title and fix some tests indention

bcba746

Skip recipient api test for swagger issue

3fb8bc2

Fix tests

7580d7b

Move tests and add blood types todos

dc5a69e

tomaspavlin dismissed kubantjan’s stale review via dc5a69e February 19, 2021 14:49

Merge remote-tracking branch 'origin/master' into 418_improve_updatin…

3c35cc8

…g_of_patients

tomaspavlin requested a review from kubantjan February 19, 2021 15:19

kubantjan previously approved these changes Feb 21, 2021

View reviewed changes

misslecter reviewed Feb 21, 2021

View reviewed changes

tomaspavlin added 2 commits February 22, 2021 11:04

Copy deep antigens

b36d852

tomaspavlin dismissed kubantjan’s stale review via 9eeb8d8 February 22, 2021 10:11

tomaspavlin requested a review from misslecter February 22, 2021 10:37

misslecter approved these changes Feb 22, 2021

View reviewed changes

tomaspavlin merged commit 89f413a into master Feb 22, 2021

tomaspavlin deleted the 418_improve_updating_of_patients branch February 22, 2021 10:39

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Improve updating of patients #444

Improve updating of patients #444

tomaspavlin commented Feb 12, 2021 •

edited

Loading

kubantjan commented Feb 19, 2021

kubantjan left a comment

tomaspavlin commented Feb 19, 2021 •

edited

Loading

kubantjan commented Feb 19, 2021

tomaspavlin commented Feb 19, 2021

kubantjan left a comment •

edited

Loading

misslecter left a comment

Improve updating of patients #444

Improve updating of patients #444

Conversation

tomaspavlin commented Feb 12, 2021 • edited Loading

BE

FE

Known bugs

kubantjan commented Feb 19, 2021

kubantjan left a comment

Choose a reason for hiding this comment

tomaspavlin commented Feb 19, 2021 • edited Loading

kubantjan commented Feb 19, 2021

tomaspavlin commented Feb 19, 2021

kubantjan left a comment • edited Loading

Choose a reason for hiding this comment

misslecter left a comment

Choose a reason for hiding this comment

tomaspavlin commented Feb 12, 2021 •

edited

Loading

tomaspavlin commented Feb 19, 2021 •

edited

Loading

kubantjan left a comment •

edited

Loading