medic / cht-core

The CHT Core Framework makes it faster to build responsive, offline-first digital health apps that equip health workers to provide better care in their communities. It is a central resource of the Community Health Toolkit.
https://communityhealthtoolkit.org
GNU Affero General Public License v3.0
469 stars 217 forks source link

fix(#8806): merge extra validations with pupil so rules can have both #9602

Closed garethbowen closed 2 weeks ago

garethbowen commented 3 weeks ago

Description

8806

8402

Code review checklist

Compose URLs

If Build CI hasn't passed, these may 404:

License

The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.

garethbowen commented 3 weeks ago

@1yuv Would you mind reviewing this? There are two regressions commented in the PR but I don't think anyone's going to notice. If you could run some of your configs through this update to confirm that would be ideal!

1yuv commented 3 weeks ago

I tested this with few configurations and I can combine multiple validations easily and is working perfectly.

1yuv commented 2 weeks ago

There's an #9604 I've filed that was discovered while testing this, but is not caused by this change.