Skip to content

Conversation

@shivvani-r
Copy link

Description of what I changed

I have added the "OpenMRS" Concept Source and introduced standard mappings for the boolean concepts (TRUE/FALSE) in the Reference Application configuration. I also updated the global properties to reflect these new mappings.

Specific changes include:

  • Added conceptsources.csv to register the official "OpenMRS" concept source (UUID: 913e06c5-4ca1-46e3-9ecf-afbe11bbd0d9).
  • Added conceptreferenceterms.csv to define the terms TRUE and FALSE under the OpenMRS source.
  • Added concepts.csv to map the core boolean concepts (UUIDs 1065... and 1066...) to OpenMRS:TRUE and OpenMRS:FALSE.
  • Updated globalproperties.xml to deprecate concept.true and concept.false properties, updating their descriptions to reference the new mappings.
  • Verified the configuration by deploying to a local OpenMRS instance and confirming that the Initializer module loaded the mappings successfully.

Issue I worked on

see https://issues.openmrs.org/browse/TRUNK-6232

Checklist: I completed these to help reviewers :)

  • My IDE is configured to follow the code style of this project.

  • I have verified the changes locally. (Verified by running the server and checking logs for successful concept mapping loading)

  • I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.

  • All new and existing tests passed.

  • My pull request is based on the latest changes of the master branch.

@shivvani-r
Copy link
Author

@dkayiwa could you please review this when you have a moment?

@dkayiwa
Copy link
Member

dkayiwa commented Jan 26, 2026

Thank you so much @shivvani-r for your contribution!
Do you mind starting a new Talk post about this and sharing your approach to solving the reported problem. Specifically call out Andrew Kanter who manages the CIEL dictionary to get his feedback on the talk post.

@shivvani-r
Copy link
Author

Thank you so much @shivvani-r for your contribution! Do you mind starting a new Talk post about this and sharing your approach to solving the reported problem. Specifically call out Andrew Kanter who manages the CIEL dictionary to get his feedback on the talk post.

Thank you for the suggestion. I'd be happy to.

@shivvani-r
Copy link
Author

@dkayiwa I have created the topic here and tagged Andrew Kanter for his feedback as suggested. Thanks!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants