Project: Complete UI rewrite or radiology and toolkit as an open web application

Sorry for that, won’t happen again.

@sunbiz Should I also separate the Patient CRUD Merge request into smaller bits, Patient Search, Patient Dashboard, Patient Update… say?

Those parts are already in the lh-toolkit-webcomponents. No point reinventing the wheel. If you have any updates to it, then please send those to that repo. You should create necessary issues under that project and submit MRs to that project to update those webcomponents for patient search, obs, etc.

Yes, I am using the web components to build the UI for the radiology and toolkit. I am not talking about updating the web components. The MRs I am talking about is the implementation of these functionalities using the web components. I feel like we are having a communication gap. Please clarify. @sunbiz

oh yea, for each page/workflow that you are implementing using the webcomponents, please create issues for them and include the issue in your commit message.

As for improving communication, I recommend that simple things like these could be done over Chat and I am usually hanging out there on most days.

Okay, that makes sense. I will keep you updated with the MRs on the chat. This week I am working on Appointments resourceType and will add MRs for Appointments as well as Patient Workflows.

@sunbiz Here is the list of MRs to be merged in order

  1. Remove Angular app and Create React App (!4) · Merge requests · LibreHealth / LibreHealth Radiology / lh-radiology-owa · GitLab
  2. Add Navbar and React Router + CI and CD (!6) · Merge requests · LibreHealth / LibreHealth Radiology / lh-radiology-owa · GitLab
  3. Implemented the `/api/[resourceType].ts` data fetching using React Query (!5) · Merge requests · LibreHealth / LibreHealth Radiology / lh-radiology-owa · GitLab
  4. Create a Patient Dashboard with a dynamic route which links to the patient Search (!7) · Merge requests · LibreHealth / LibreHealth Radiology / lh-radiology-owa · GitLab
  5. Add lh-toolkit-webcomponents and implement Patient Demographics (!8) · Merge requests · LibreHealth / LibreHealth Radiology / lh-radiology-owa · GitLab
  6. Implement Update Patient and Create Patient (!9) · Merge requests · LibreHealth / LibreHealth Radiology / lh-radiology-owa · GitLab

You are not blogging weekly. That’s an expectation that was laid out as an expectation in the beginning.

I’ll start blogging from now on. Sorry for not blogging consistently.

@sunbiz As you mentioned that I will have to write new web components in lh-toolkit-webcomponents for all the Radiology Stuff, and the /radiologyxxx will be available can you please provide more information as to how the app authenticates to make use of these API endpoints? And how can I develop for these API end points on my local machine?

As for the web components I am going through LibreHealth Toolkit - Home to dissect the UI into small parts which can be made into web components and added to lh-toolkit-webcomponents

Reminder about the Weekly Status Report

Another Reminder about the Weekly Status Report

I should not have to track you down. Our expectations were laid out clearly in the beginning.

The endpoints for radiology:

Requires authentication, and I would need to incorporate that into our application but I don’t know how authentication works on radiology.librehealth.io

Apart from this, I would need to write new web components for radiology and get them merged in lh-toolkit-webcomponent so I wanted to ask will those component also follow the same data architecture as the APIs on radiology.librehealth.io

The authentication is the same way the toolkit authentication works, since it is only a module on toolkit.

Your weekly report is due.

Okay, can you please redirect how I can incorporate that into my development workflow so that I can start writing web-components?

@sunbiz @r0bby Should I prefix the components with fhir as it is for all the other components for radiology components, in lh-toolkit-components too?

For example: fhir-radiology-filter

Yes, I think that’s a good naming convention

1 Like

lh-fhir-radiology-filter is my preference

1 Like

This week is your last weekly report.

1 Like