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

Updates to the visit admission/discharge functionality #5232

Open
jmcameron opened this issue Dec 10, 2020 · 1 comment
Open

Updates to the visit admission/discharge functionality #5232

jmcameron opened this issue Dec 10, 2020 · 1 comment

Comments

@jmcameron
Copy link
Collaborator

jmcameron commented Dec 10, 2020

  1. [OBSOLETE] In the patient record, if the user clicks on the [Discharge Patient] button, they see a list of possible discharge reasons (from the discharge_type table). If this table is empty, the "Type of discharge" header should be omitted.

  2. In the visits registry, we should add "Discharge" items under the action menu for patients who are currently admitted (or a "Admit" for those nut currently admitted. These could also be added to the patient registry.

  3. In the Visits Registry, we should add a filter for patients who are currently admitted (not discharged yet)

  4. [DEFER] Do we have a way to create/edit Discharge types from the Bhima interface? Not sure that this makes sense since these are translation tokens which are translated into the current language. Obviously we could add a create/edit function for the discharge_type table, but this would not address the corresponding language tokens. It seems to me that adding a database item for these types of translation items is inefficient. We should just have a data structure somewhere in the Bhima code that has the id, and label for each discharge type. Then adding a new item would be strictly done via a code change. Now we have to do a code change and a database update. This could all be done via database tables but would require different approach since table entries would be needed for each language.

@jmcameron
Copy link
Collaborator Author

jmcameron commented Jan 11, 2021

Based on PR #5275, we will update all sites with the default list of discharge types (see server/models/bhima.sql). So Item 1 is no longer needed.

Although item 4 would be nice (ability for end users to create new discharge types), it would require a significant refactor and should be deferred. For the foreseeable future, to add a new discharge type will continue to require an SQL update as well as adding new items to the i18n translation files.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants