feat(#566): implement auto-inclusion of task and target and contact s…#784
feat(#566): implement auto-inclusion of task and target and contact s…#784inromualdo wants to merge 6 commits intomainfrom
Conversation
|
@inromualdo sorry for the delay! Very interested in the feature you have added here! Before I get too far into reviewing the code, can you help me understand a bit better how this is expected to work? Here is how I think things go and maybe you can correct me if I get anything wrong:
There are some tradeoffs here between clarity and convenience. (There is more complexity when we are merging files behind the scenes and it adds another level of abstraction between what is seen in the config files and what is actually uploaded to the server. On the other hand, making configuration more reusable has clear benefits as well.) I think I see how this works for just dropping new files into the project directory. How will this work for config coming from re-usable NPM libs like https://github.com/medic/cht-stock-monitoring-workflow? Would I still have to manually copy files from that repo into my local config? |
Description
Ability to compile and upload tasks and contact summary from reusable componnent. This PR auto include any file end by
.tasks.js,.targets.jsand.contact-summary.js#566
Code review items
License
The software is provided under AGPL-3.0. Contributions to this project are accepted under the same license.