Skip to content

Conversation

@kaapstorm
Copy link
Contributor

🦖 Jira: SAAS-19012

  • Moves documentation into the docs/ directory
  • Adds documentation to build a data pipeline from CommCare to DHIS2

Copy link
Contributor

@millerdev millerdev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small comment, I think non-blocking.

Comment on lines +32 to +34
See the
[**commcare-sync-ansible** documentation](https://commcare-sync-ansible.readthedocs.io/en/latest/)
for details on installing and managing a production environment.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks like a near duplicate of docs/install_prod.md. Will it be confusing to have both?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

My thinking was that (1) the documentation in docs/ is reference material, and should include all the ways that users can install, and so it should refer readers to the docs for production. And that (2) README.md is an overview that can point people quickly to what they most likely want next, and that could be either developer docs or production docs.

If you don't think that makes sense, or if you think the wording in README.md should be tweaked, I'd be happy to do that. I'll merge for now, and we can follow up in a second, small PR.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense, thanks for explaining.

@kaapstorm kaapstorm merged commit 17d50ff into master Jan 2, 2026
2 checks passed
@kaapstorm kaapstorm deleted the nh/docs branch January 2, 2026 10:43
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.

3 participants