Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

commcare: update examples documentation #596

Merged
merged 2 commits into from
Nov 4, 2024
Merged

Conversation

hunterachieng
Copy link
Contributor

@hunterachieng hunterachieng commented Nov 4, 2024

Short Description

Updated examples on commcare

Closes #775

Details

New examples on bulk and submitXls have been added as a guide on possible usage

AI Usage

Please disclose how you've used AI in this work (it's cool, we just want to
know!):

  • Code generation (copilot but not intellisense)
  • Learning or fact checking
  • Strategy / design
  • Optimisation / refactoring
  • Translation / spellchecking / doc gen
  • Other
  • I have not used AI

You can read more details in our
Responsible AI Policy

@@ -0,0 +1,8 @@
// Update the kunjungan case with a new row of data. The row contains the case_id and satusehat_id

submitXls([{ case_id: 'abcde', satusehat_id: '12345' }], {
Copy link
Contributor

Choose a reason for hiding this comment

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

Since this is just a one-liner, would it be better if it were just a regular example in the adaptor docs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, its already in the adaptor docs Let me remove it

Signed-off-by: Hunter Achieng <[email protected]>
@josephjclark josephjclark merged commit d8321a7 into main Nov 4, 2024
1 check passed
@josephjclark josephjclark deleted the feat/775-commcare-docs branch November 4, 2024 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants