Skip to content

feat(#9835): add cht datasource apis for creation and update of contacts and reports - #10083

Merged
sugat009 merged 53 commits into
masterfrom
9835-add-cht-datasource-apis-for-creation-and-update-of-contacts-and-reports
Mar 10, 2026
Merged

sugat009 merged 53 commits into
masterfrom
9835-add-cht-datasource-apis-for-creation-and-update-of-contacts-and-reports

Conversation

@sugat009

@sugat009 sugat009 commented Jun 24, 2025 •

Copy link
Copy Markdown
Member

Description

Adds APIs for creating and updating contacts and reports.

Ticket: #9835

Code review checklist

  • UI/UX backwards compatible: Test it works for the new design (enabled by default). And test it works in the old design, enable can_view_old_navigation permission to see the old design. Test it has appropriate design for RTL languages.
  • Readable: Concise, well named, follows the style guide, documented if necessary.
  • Documented: Configuration and user documentation on cht-docs
  • Tested: Unit and/or e2e where appropriate
  • Internationalised: All user facing text
  • Backwards compatible: Works with existing data and configuration or includes a migration. Any breaking changes documented in the release notes.

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.

apoorvapendse and others added 10 commits June 5, 2025 13:21
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Co-authored-by: Sugat Bajracharya <sugatbajracharya49@gmail.com>
…10022)

Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: apoorvapendse <apoorvavpendse@gmail.com>
Signed-off-by: apoorvapendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Co-authored-by: Sugat Bajracharya <30311933+sugat009@users.noreply.github.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Co-authored-by: Sugat Bajracharya <30311933+sugat009@users.noreply.github.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: apoorvapendse <apoorvavpendse@gmail.com>
Co-authored-by: Sugat Bajracharya <30311933+sugat009@users.noreply.github.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: apoorvapendse <apoorvavpendse@gmail.com>
Co-authored-by: Sugat Bajracharya <30311933+sugat009@users.noreply.github.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Co-authored-by: Sugat Bajracharya <30311933+sugat009@users.noreply.github.com>
@sugat009
sugat009 requested a review from jkuester June 24, 2025 12:52
@sugat009 sugat009 linked an issue Jun 24, 2025 that may be closed by this pull request
6 tasks done
@sugat009 sugat009 added the Type: Feature Add something new label Jun 24, 2025
@sugat009 sugat009 moved this to 💻 In Progress in CHT Community Contributions Q2 2025 Jun 24, 2025
@sugat009 sugat009 self-assigned this Jun 24, 2025
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
@sugat009 sugat009 assigned sugat009 and unassigned sugat009 Jun 25, 2025
Signed-off-by: apoorvapendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Comment thread shared-libs/cht-datasource/src/qualifier.ts Outdated
Comment thread shared-libs/cht-datasource/src/qualifier.ts Outdated
Comment thread shared-libs/cht-datasource/src/qualifier.ts Outdated
Signed-off-by: apoorvapendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
Signed-off-by: Apoorva Pendse <apoorvavpendse@gmail.com>
@andrablaj andrablaj removed the Stale label Jan 22, 2026
…and-update-of-contacts-and-reports

# Conflicts:
#	tests/integration/api/controllers/contact.spec.js
#	tests/integration/api/controllers/person.spec.js
#	tests/integration/api/controllers/place.spec.js
#	tests/integration/api/controllers/report.spec.js
#	tests/integration/shared-libs/cht-datasource/contact.spec.js
#	tests/integration/shared-libs/cht-datasource/person.spec.js
#	tests/integration/shared-libs/cht-datasource/place.spec.js
#	tests/integration/shared-libs/cht-datasource/report.spec.js
@github-actions

Copy link
Copy Markdown

This PR is now marked "stale" after 30 days without activity. It will be closed automatically in 10 days unless you add a comment, push new changes or remove the "stale" label.

@github-actions github-actions Bot added the Stale label Feb 26, 2026
@sugat009 sugat009 removed the Stale label Feb 26, 2026
jkuester and others added 11 commits March 3, 2026 17:12
Co-authored-by: Sugat Bajracharya <30311933+sugat009@users.noreply.github.com>
…and-update-of-contacts-and-reports

# Conflicts:
#	shared-libs/cht-datasource/src/index.ts
#	shared-libs/cht-datasource/src/local/contact.ts
#	shared-libs/cht-datasource/src/local/libs/core.ts
#	shared-libs/cht-datasource/src/local/libs/doc.ts
#	shared-libs/cht-datasource/src/local/report.ts
#	shared-libs/cht-datasource/test/local/contact.spec.ts
#	shared-libs/cht-datasource/test/local/libs/core.spec.ts
#	shared-libs/cht-datasource/test/local/libs/doc.spec.ts
#	shared-libs/cht-datasource/test/local/report.spec.ts
#	shared-libs/cht-datasource/test/person.spec.ts
#	shared-libs/cht-datasource/test/report.spec.ts
#	tests/integration/api/controllers/person.spec.js
#	tests/integration/api/controllers/place.spec.js
#	tests/integration/api/controllers/report.spec.js
#	tests/integration/shared-libs/cht-datasource/person.spec.js
#	tests/integration/shared-libs/cht-datasource/place.spec.js
#	tests/integration/shared-libs/cht-datasource/report.spec.js

@jkuester jkuester left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Okay, @sugat009, I think this is good to go! 🚀 🚀 You might give it one more check over to make sure I did not break anything when pulling in the updates from master.

The e2e tests are struggling, but I think this is from master and not related to our changes here. (Diana said in the Dev Hour today that she was working on fixes for the e2e tests in a separate PR...)

Once again, huge shout-out to @apoorvapendse for all your work on this project! It has been a long time in coming, but I think we have finally made it! 🎉

@apoorvapendse

Copy link
Copy Markdown
Contributor

Great!!
Thank you for taking this to completion @jkuester, @sugat009.

I just flipped through this gist and it brought back a lot of happy memories.
I miss the dev hours and the fun discussions we had during our weekly syncs and on the forum.
I'll try to show up sometime if the dev hour is still a thing.

Thank you once again for the C4GT experience and for integrating this.
@dianabarsan, @andrablaj , @antonykhaemba, @sugat009, @jkuester, @lorerod and all the other wonderful folks I met during that time; I hope you have a great year ahead and all the best for the upcoming C4GT season!

@sugat009

Copy link
Copy Markdown
Member Author

I reviewed and tested this. We can merge it to master. The docs PR needs to be updated as well with the changes in permissions. Working on it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: 💻 In Progress

Development

Successfully merging this pull request may close these issues.

Add cht-datasource APIs for creation and update of contacts and reports

4 participants