medic / cht-docs

Documentation site for the Community Health Tookit

https://docs.communityhealthtoolkit.org

14 stars 16 forks source link

chore(#1405): Update CHT Sync architecture #1429

Closed andrablaj closed 1 month ago

andrablaj commented 1 month ago

Description

Following the architecture changes in CHT Sync, this PR updates the existing documentation to match the new system design approach.

Closes #1405

License

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

mrjones-plip commented 1 month ago

oops! i'm too late for the question with the PR being merged - but I did want to ask anyway: do we need these two lines?

Install the dependencies:
 npm ci

I ask because the entire page has docker compose calls and no other node or npm calls which makes me think maybe we don't need it?

andrablaj commented 1 month ago

Great catch, @mrjones-plip! That command is not necessary for running docker compose (I just checked locally to be extra sure). I opened a small PR to remove it.