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 15 forks source link

GAPS: Demographics must be linked to year. #4147

Closed jmbrunskill closed 3 weeks ago

jmbrunskill commented 5 months ago

What went wrong? 😲

Because we have decided to records demographic against a baseYear to avoid confusion about when population growth is applied too, we now only have the option of creating demographic records against a particular year.

image

Expected behaviour 🤔

When selecting a demographic in an immunisation program, it should be independant of the year.

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Login to Central OMS (with module:vaccine enabled)

  2. Click on Demographics

  3. Create a new demographic indicator image

  4. Click on Program -> Immunisation

  5. Create a Program, and Vaccine Course

  6. Try to select a demographic (see the year is included)

Your environment 🌱

Agreed solution

mark-prins commented 1 month ago

Refinement: would like to proceed and the approach sounds valid. up to scheduling decisions now

lache-melvin commented 3 weeks ago

Closed by #5126

marthakeezy commented 2 weeks ago

Tested in V2.3.1-RC1, all working as expected :) Done ✅