openmrs-module-initializer icon indicating copy to clipboard operation
openmrs-module-initializer copied to clipboard

Published 20 hours ago • verified publisher icon mekomsolutions

Support explicit `addresshierarchy` domain.

Open mks-d opened this issue 1 year ago • 3 comments

Introduce an addresshierarchy (AH) domain that takes care of:

  1. Load all AH-related configs (address template, address entries, etc).
  2. Filling the Ext I18n's i18n reverse cache.

This work will demand for Iniz to become aware of AH:

  • Let's release AH 2.15.3 and bump it to AH 3.0.0.
    • Iniz will depend and be aware of AH 3.0.0.
  • AH 3.0.0 will not support self-loading of AH configs anymore.
    • Let's create an ADDR ticket for this ↑ non-backward compatible change.

mks-d avatar Oct 16 '23 16:10 mks-d

If we are removing a feature like this from AH, then we should bump the version to 3.0.0, not just up to 2.16.0.

mseaton avatar Oct 16 '23 16:10 mseaton

Thanks @mseaton and for the exchange on Slack, I have updated the description accordingly.

mks-d avatar Oct 16 '23 16:10 mks-d

I've created ADDR-129 to followup the necessary refactoring on the AH module.

Ruhanga avatar Oct 30 '23 10:10 Ruhanga

@Ruhanga was this not completed in the end? If it is, could you close this PR? → https://github.com/mekomsolutions/openmrs-module-initializer/pull/188

mks-d avatar Jun 28 '24 14:06 mks-d