Skip to content

Add debug logging#1763

Open
MartinRiese wants to merge 1 commit intomasterfrom
riese/log_4_hashes
Open

Add debug logging#1763
MartinRiese wants to merge 1 commit intomasterfrom
riese/log_4_hashes

Conversation

@MartinRiese
Copy link
Contributor

Product Description

We have a bug with case search and want to know how prevalent it is in production. On the BHA domains in manifests with the first column in the case search results being "####".

Technical Summary

https://dimagi.atlassian.net/browse/USH-6370

Safety Assurance

Safety story

Tested locally with a different domain name.
Only add logging.

Automated test coverage

None

QA Plan

None

Special deploy instructions

  • This PR can be deployed after merge with no further considerations.

Rollback instructions

  • This PR can be reverted after deploy with no further considerations.

Review

  • The set of people pinged as reviewers is appropriate for the level of risk of the change.

@MartinRiese MartinRiese marked this pull request as ready for review February 11, 2026 16:26
@MartinRiese MartinRiese requested a review from esoergel February 11, 2026 16:27
@codecov
Copy link

codecov bot commented Feb 11, 2026

Codecov Report

❌ Patch coverage is 7.14286% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.21%. Comparing base (969757f) to head (c86a250).

Files with missing lines Patch % Lines
...ommcare/formplayer/application/MenuController.java 7.14% 12 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1763      +/-   ##
============================================
- Coverage     70.36%   70.21%   -0.16%     
+ Complexity     2032     2029       -3     
============================================
  Files           257      257              
  Lines          7995     7996       +1     
  Branches        755      761       +6     
============================================
- Hits           5626     5614      -12     
- Misses         2087     2099      +12     
- Partials        282      283       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@esoergel esoergel left a comment

Choose a reason for hiding this comment

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

Looks good to me, depending on resolution of the sentry notifications. Given the importance of this code, I'd for sure want to test this out on staging first.

We have a bug with case search and want to know how prevalent it is
in production. On the BHA domains in manifests with the first column
in the case search results being "####".
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