search

msupply-foundation / open-msupply

Open mSupply represents our most recent advancement in the Logistics Management Information System (LMIS), expanding on more than two decades of development inherited from the well-established legacy of the original mSupply.
https://msupply.foundation/open-msupply/
Other
23 stars 14 forks source link

Able to set a visit date prior to patient's birth date #5219

Open marthakeezy opened 1 month ago

marthakeezy commented 1 month ago

What went wrong? 😲

In encounters, the user is able to set visit date prior to patient's birth date.

https://github.com/user-attachments/assets/2916cc68-13b5-48e1-988a-1218451b2e39

Expected behaviour 🤔

Should be some sort of restriction -> thinking how you cannot backdate prescriptions to the date when the item didn't exist in the system, similarly visit date should not be able to be selected prior to one's date of birth.

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Have a patient who is enrolled in a Program
  2. Have their DOB be something recent
  3. Create an encounter for that patient and try and change the date to before their DOB
  4. The system allows that action -> inconsistency

Your environment 🌱

myspace20 commented 1 month ago

I will like to work on this.

roxy-dao commented 1 month ago

Welcome @myspace20! Great to see that you're interested in contributing!

For some general guidelines, check out our contributing doc.

To get started with this issue, you'll need to be working with the react app. After cloning the repo, head over to the read me which walks you through getting set up. Let me know if you need some help!

myspace20 commented 1 month ago

@roxy-dao I'm facing an issue when i try to login into the app. The issue is this : "Your account does not have access to any stores on this site". I'm using the Admin credentials from the Users.txt file.

roxy-dao commented 3 weeks ago

@roxy-dao I'm facing an issue when i try to login into the app. The issue is this : "Your account does not have access to any stores on this site". I'm using the Admin credentials from the Users.txt file.

Hi @myspace20. This has recently been fixed. Can you pull from latest develop and try again?

myspace20 commented 3 weeks ago

Yes It works now.

myspace20 commented 3 weeks ago

@roxy-dao please if you have some time to spare, let me know so that our can give me a little bit more of technical information regarding the project. I will I appreciate it.

roxy-dao commented 2 weeks ago

@roxy-dao please if you have some time to spare, let me know so that our can give me a little bit more of technical information regarding the project. I will I appreciate it.

Sorry it took me so long to get back to you! I have just realised that the export does not contain information for the module needed to fix this issue. I apologise for not realising this sooner.

There are still plenty of other issues you can do if you still wish to contribute to Open mSupply! You can filter out good first issues to find them all! (:

myspace20 commented 2 weeks ago

Thanks I will do that. But do check the sync configuration with central and test servers. It refuses connections on port 2048.

andreievg commented 4 days ago

Triage: Needs to be consolidated in a general program fix issue, so that we can reduce issue count and look at all of the issues at once