[WIP] feat(patient) secondary, tertiary addresses, phone numbers, and emails

[sourabbanka22]

A Feature that adds the ability to define secondary, tertiary, etc. phone numbers, addresses, and
emails for a patient.

New Feature #1920
Fixes #1920 .

Below are some Screenshots of the Added Functionality:
Before Adding PhoneNumber and Email of a Patient:

After Adding PhoneNumber and Email:

User needs to click on the Add Button to add phone numbers, emails, etc to the respective array.

Currently I have commented out few tests related to Phone Number, Email and Address Models

Please let me know if there are any Modifications or Changes required.

[gitpod-io]

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/hospitalrun/hospitalrun-frontend/ckm8w3x1l
✅ Preview: https://hospitalrun-fron-git-fork-sourabbanka22-secondary-tertia-873150.hospitalrun.now.sh

[jackcmeyer]

Design changes:

Create Patient

• When creating a new patient, I think that the everything should be inside the table rather than having a table only for the already added phone numbers. So, the page loads with the email, phone number, etc tables already loaded and there are empty boxes inside the table. (I hope that makes sense).
• I think the button should be on the bottom right side of the table since that is the order a user would work in
• I think the buttons should say "Add Email, "Add Phone Number", etc.

View Patient

• When viewing a patient, there should be no add buttons. Only the tables should be seen here.

Both

• I think the tables should render inside a panel that better distinguishes what the table means. The heading of the panel should be "Address", "Email", etc.

[sourabbanka22]

@jackcmeyer I have made the design changes that you suggested

Please let me know if there are any Modifications or Changes required.

[jackcmeyer]

Some more design changes:

• I think the button format should be in the link styling to be a little smaller.
• The button should be outside of the table in the of the table, that way the table content takes up the whole panel.

[jackcmeyer]

Sorry for the number of review comments, mostly around:

• duplicate code
• internationalization of keys
• we'll need to get tests added for the new functionality, plus update any existing tests

Please let me know anything I can do to help!

[matteovivona]

@sourabbanka22 tests have dropped too far, exceeding our maximum limit to accept a PR. I'd ask you to check it out and fix it.

[sourabbanka22]

@tehkapa I am working on fixing the tests

[sourabbanka22]

Hi @kumikokashii , I have created component instances for Phone Numbers, Emails and Addresses, does the code look good to you?

@tehkapa I have added all the required tests for the Added Functionalities, please let me know if there is anything else that I can help with

[ghost]

@sourabbanka22 Thank you for the updates 🙏 It's looking almost ready to go. I'm gonna jump in and tweak it a bit. Please stay tuned. Thanks again!

[ghost]

In addition to allowing more than one phone number, etc, updating the GeneralInformation component to have its onChange prop to pass the entire patient instead of a key and a value. Due to high volume of changes, transitioning to another branch. Please see #2113.