Support custom providerName for Patients

[arkadiuszbachorski]

Support custom providerName for Patients

♻️ Current situation & Problem

Closes #103

⚙️ Release Notes

• Support custom providerName for Patients

✅ Testing

Tested manually

⚠️ Do not merge this until models are updated to the latest version.

Code of Conduct & Contributing Guidelines

By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

• I agree to follow the Code of Conduct and Contributing Guidelines.

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 17 lines in your changes missing coverage. Please review.

Project coverage is 0.88%. Comparing base (5f72377) to head (0b59b79).
Report is 6 commits behind head on main.

Files with missing lines	Patch %	Lines
routes/~_dashboard/~patients/PatientForm.tsx	0.00%	15 Missing ⚠️
routes/~_dashboard/patients/$id/~index.tsx	0.00%	1 Missing ⚠️
routes/~_dashboard/~patients/~invite.tsx	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##            main    #105      +/-   ##
========================================
- Coverage   0.89%   0.88%   -0.01%     
========================================
  Files         75      76       +1     
  Lines       3711    3793      +82     
  Branches      84      85       +1     
========================================
  Hits          33      33              
- Misses      3607    3688      +81     
- Partials      71      72       +1     

Files with missing lines	Coverage Δ
routes/~_dashboard/patients/$id/~index.tsx	0.00% <0.00%> (ø)
routes/~_dashboard/~patients/~invite.tsx	0.00% <0.00%> (ø)
routes/~_dashboard/~patients/PatientForm.tsx	0.00% <0.00%> (ø)

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c1c16b...0b59b79. Read the comment docs.

[PSchmiedmayer]

Thank you for working on this @arkadiuszbachorski.
Only thing missing is the fact that the Docker compose build seems to fail.

@pauljohanneskraft let's merge this once we have published the latest version of the models package, probably also including the disabled flag.

[arkadiuszbachorski]

Only thing missing is the fact that the Docker compose build seems to fail.

@PSchmiedmayer it is going to fail, because types are incorrect. I used latest version of models package locally and this PR tries to rely on models which do not have providerName. It's going to be resolved once all the firebase PRs are merged and new models version is published.

[pauljohanneskraft]

Looks good to me - except for one small non-blocking question.

[PSchmiedmayer]

Sounds great!
Thank you @arkadiuszbachorski & @pauljohanneskraft for merging all the changes in the Firebase repo and merging this PR, as far as I can see we can tag a 0.6.0 release in the Firebase repo 👍