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.

Sign up for GitHub

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

Jump to bottom

Deploy patient list #23

Merged
merged 24 commits into from
Jul 20, 2021
Merged

Deploy patient list #23

merged 24 commits into from
Jul 20, 2021

Conversation

FlorianRappl
Copy link
Contributor

See previous PR: #22

What does this PR do?

This PR comprises of various components:

  1. Modal Design
  2. Fetching patient lists
  3. Review changes (reviews given to Deploy patient list #19 )
  4. Implementing search for patient lists.

Design: https://app.zeplin.io/project/60d59321e8100b0324762e05/screen/60d5993c822cc112375b4379

nmalyschkin and others added 23 commits June 15, 2021 15:46
@nmalyschkin
add offline patient list data
650942f
@nmalyschkin
review improvements
5665e76
@nmalyschkin
add patient list api, add patient list action button to patient chart
aa65986
@nmalyschkin
remove console.log
989e1bb
@nmalyschkin
add modal for adding patients to lists
7929d69
@nmalyschkin
Merge remote-tracking branch 'origin/main' into deploy_patient_list
e33359b
@vasharma05
Design completed
a0f4fd9
@nmalyschkin
start replacing the mock with the API
3594a79
@vasharma05
Searching patient lists added
b742cba
@nmalyschkin
replace mock with API in hooks
fc94e59
@vasharma05
Merge branch 'deploy_patient_list' of https://github.com/openmrs/open…
f125f12 
…mrs-esm-patient-management into deploy_patient_list
@vasharma05
Change in structure of PatientListData
39fb226
@vasharma05
Selecting and adding patient to list implemented
00922bf
@vasharma05
MF-641: Add Patient List Modal (openmrs#18)
4e46ae5 
* Design completed

* Searching patient lists added

* Change in structure of PatientListData

* Selecting and adding patient to list implemented
@nmalyschkin
fix addPatient, add function to retrieve lists for a patient
df940d1
@nmalyschkin
fix API
e1c2e78
@nmalyschkin
Merge remote-tracking branch 'origin/main' into deploy_patient_list
c8b5761
@nmalyschkin
fix getPatientListsForPatient method
a477556
@nmalyschkin
update yarn.lock
54eaa5d
@vasharma05
Merge branch 'deploy_patient_list' of https://github.com/openmrs/open…
0816439 
…mrs-esm-patient-management into deploy_patient_list
@vasharma05
Review changes
a084cec
@vasharma05
Implemented search
cef9b0c
@vasharma05
Hiding the already enrolled lists completed
3d6883b
@FlorianRappl
Copy link
Contributor Author

Hi @vasharma05 can you take care of merging this properly? Sorry, the PR from Nikita was merged first (I did not notice you had a PR against another satellite branch...).

@FlorianRappl
Copy link
Contributor Author

Hi @vasharma05 - do you have this on the radar? Thanks!

@vasharma05
Copy link
Member

Hi Florian! I have this on the list, will be pushing the new tomorrow morning asap. Thanks!

@vasharma05
Copy link
Member

@FlorianRappl I have cleared the merge conflicts.

@FlorianRappl FlorianRappl merged commit e830ad1 into openmrs:main Jul 20, 2021
@vasharma05 vasharma05 deleted the deploy_patient_list branch August 5, 2021 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@FlorianRappl @vasharma05 @nmalyschkin