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

🛠️ Replace useDispatch w. useQuery/request: Patient (src/Components/Patient/**) #6546

Closed
Tracked by #6316
rithviknishad opened this issue Nov 6, 2023 · 8 comments · Fixed by #7078
Closed
Tracked by #6316
Assignees
Labels
good first issue Good for newcomers

Comments

@rithviknishad
Copy link
Member

No description provided.

@konavivekramakrishna
Copy link
Contributor

konavivekramakrishna commented Nov 6, 2023

I would like work on this, can you please assign me

@rithviknishad
Copy link
Member Author

rithviknishad commented Nov 6, 2023

@konavivekramakrishna You've 2 issues assigned to you already that are not completed yet (1 has no PR, 1 has changes requested to be followed up)

https://github.com/coronasafe/care_fe/issues/assigned/@me

Feel free to take up the issues once they are completed.

@krushn-a
Copy link

krushn-a commented Nov 6, 2023

Hi @rithviknishad, I would like to work on this issue.

@rithviknishad rithviknishad moved this from Triage to Up Next in Care Nov 6, 2023
@krushn-a
Copy link

krushn-a commented Nov 9, 2023

Hi @rithviknishad from the past 3 days i was going through the files of patient folder and some references related to this type of issue, I am facing a bit problem that is i couldn't able to understand how could i come up with code by which i can replace usedispatch() usage from files. So please if you could help me with me it...

@rithviknishad
Copy link
Member Author

rithviknishad commented Nov 9, 2023

Hey @krushn-a, feel free to look at the documentation #6316 and the example PRs attached in the same issue.

This issue will have 17 files to be changed. Feel free to break this issue further down to multiple sub-tasks, and have multiple PR's so that it'd be easier to review and test (also to steer away from having merge conflicts). Have around 1-3 files changed at a time in a single PR.

@rithviknishad
Copy link
Member Author

Hey @krushn-a
Were you able to get back at this? Are you facing any blockers?

@krushn-a
Copy link

I working on it and am really trying my best. Just need some more time.

@konavivekramakrishna
Copy link
Contributor

konavivekramakrishna commented Jan 18, 2024

@rithviknishad can you please assign this to me

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants