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

Clean up telehealth screen code #46

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

Conversation

amlannandy
Copy link
Contributor

Description

The telehealth section code was split into multiple components to make it more readable and maintainable.

  1. Chats and Directors section were moved into separate files and now use StreamBuilder to fetch and display data.
  2. Rest of the screens and other sections were split up into separate components in the same file.

Testing Methods

N/A

Screenshots/Videos

N/A

New Packages Added

N/A

Closing Issues

closes #45

@amlannandy amlannandy changed the title Clean up telehealth scren code Clean up telehealth screen code Feb 7, 2021
@im-Amitto
Copy link
Member

I am not really sure about this one as we still have ~500 lines on that file and now we have another file that only has ~80 lines of code and this makes things hard to import. I f we are doing a split then let's do it in a production fashion.
Move constants in one, create a structure for the ones who don't have one, and add such basic improvement as well.

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.

Clean up Telehealth screen code
2 participants