Skip to content

feat(stoptb): STOP-601 remarks, STOP-650 other symptoms details, STOP-651 sputum collection date - #401

Merged
SauravBizbRolly merged 5 commits into
release-3.13.0from
vb/3.13.0-merge-main
Nov 3, 2026
Merged

SauravBizbRolly merged 5 commits into
release-3.13.0from
vb/3.13.0-merge-main

Conversation

@vishwab1

@vishwab1 vishwab1 commented Nov 3, 2026

Copy link
Copy Markdown
Member

📋 Description

JIRA ID:

Please provide a summary of the change and the motivation behind it. Include relevant context and details.


✅ Type of Change

  • 🐞 Bug fix (non-breaking change which resolves an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 🔥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 🛠 Refactor (change that is neither a fix nor a new feature)
  • ⚙️ Config change (configuration file or build script updates)
  • 📚 Documentation (updates to docs or readme)
  • 🧪 Tests (adding new or updating existing tests)
  • 🎨 UI/UX (changes that affect the user interface)
  • 🚀 Performance (improves performance)
  • 🧹 Chore (miscellaneous changes that don't modify src or test files)

ℹ️ Additional Information

Please describe how the changes were tested, and include any relevant screenshots, logs, or other information that provides additional context.

vishwab1 and others added 5 commits October 14, 2026 18:45
Store the free-text "Other symptoms" entered when Others = Yes in
tb_screening.OtherSymptomsDetails and return it from tbScreening/getAll.
Details are cleared when Others is not Yes, trimmed, and capped at 200
characters; older app versions that don't send the field still save.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…-651)

Store sputumCollectionDate (epoch ms) in
tb_stoptb_diagnostics.SputumCollectionDate and return it from
diagnostics/getAll. The date is cleared when isSputumCollected is false
and left unchanged when the field is not sent, so older app versions
keep working. The today/yesterday window is enforced in the app because
records can sync days after entry.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
OutputResponse.toString() drops null values, so sputumCollectionDate
was missing from records with no date. Use toStringWithSerializeNulls()
so the key is always present.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- STOP-650: save and return Other symptoms details in TB screening
- STOP-651: save and return Sputum Sample Collection date in diagnostics;
  diagnostics getAll keeps null fields

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Map the existing tb_confirmed_cases.remarks column. On save, remarks is
only updated when the app sends it, so older app builds don't clear it.
getAll now returns remarks.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit ab5a747)
@coderabbitai

coderabbitai Bot commented Nov 3, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 72259e42-256b-444d-a6cb-290b932e6f9d
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

CodeRabbit Verify

  • Run CodeRabbit Verify to test the changes in this pull request.

@SauravBizbRolly
SauravBizbRolly merged commit f0f375b into release-3.13.0 Nov 3, 2026
3 checks passed
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.

2 participants