Skip to content

Fix the emergency contacts and Start Call issue in 104 and 1097 - #458

Closed
vanitha1822 wants to merge 13 commits into
release-3.13.0from
nd/vs/fix_104_issues
Closed

Fix the emergency contacts and Start Call issue in 104 and 1097#458
vanitha1822 wants to merge 13 commits into
release-3.13.0from
nd/vs/fix_104_issues

Conversation

@vanitha1822

@vanitha1822 vanitha1822 commented Sep 30, 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.

vanitha1822 and others added 12 commits July 6, 2026 15:50
@vanitha1822
fix: facility id null (#433)
826c32d
@SauravBizbRolly
fix order issue in dynamic from
a552831
@vanitha1822
fix: call report issue for 104 (#445)
99b17d8 
* fix: call report issue for 104

* fix: update cti logger properties

* fix: update the properties

* fix: update the pom version

* fix: add the range to sync

* fix: update the configproperties to fetch the value from .env
@snehar-nd Sneha @claude
fix: use valid HTTP status 500 instead of invalid 5000 in report erro…
e88fe3b 
…r responses (#448)

CustomerRelationshipSecondaryReports controller returned ResponseEntity.status(5000)
on generic exceptions across all report endpoints (including getAllByGender and
getAllBySexualOrientation used by the 1097 report downloads). 5000 is not a valid
three-digit HTTP status, so Spring throws IllegalArgumentException while building
the response, which surfaces to clients as a confusing
"Status code '5000' should be a three-digit positive integer" error instead of the
actual failure reason.

Co-authored-by: Sneha <sneha@ADMINs-MacBook-Pro.local>
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
@vanitha1822
Revert the unwanted scheduler changes (#449)
d32456a 
* Revert "fix: add the range to sync"

This reverts commit 92b3f20.

* Revert "fix: call report issue for 104"

This reverts commit a47b709.
@SauravBizbRolly
Merge branch 'release-3.9.0' into bug/fix_order_issue_in_dynamic_form
3f5abdf
@SauravBizbRolly
Merge pull request #452 from PSMRI/bug/fix_order_issue_in_dynamic_form
9277c4b 
Bug/fix order issue in dynamic form
@vanitha1822
fix: remove the wrong column (#454)
71fcbb2
@vanitha1822
fix: build issue (#455)
be20463
@vanitha1822
fix: emergency contacts issue
b69b295
@vanitha1822
fix: welcome sms issue
a7a66a4
@vanitha1822
fix: merge with 3.9.0
bb10872
@coderabbitai

coderabbitai Bot commented Sep 30, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

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: 489e71a0-cc38-4acd-b80e-6d8554729d9b
  • 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

Comment @coderabbitai help to get the list of available commands.

@drtechie

drtechie commented Oct 26, 2026

Copy link
Copy Markdown
Member

Hi @vanitha1822 what is this PR about?
Assam prod deployment has issues.
We added Transient for this logger variable.

#461

But this PR has more things. Is this supposed to be merged?

@vanitha1822

vanitha1822 commented Oct 27, 2026

Copy link
Copy Markdown
Member Author

This PR addresses the issues with emergency contacts and call initiation in 104 and 1097, as reported by Aarti. These fixes are required to merge @drtechie . I will rebase and merge this.

@sonarqubecloud

sonarqubecloud Bot commented Oct 27, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
8 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@vanitha1822
vanitha1822 changed the base branch from release-3.9.1 to release-3.13.0 October 27, 2026 05:19
@vanitha1822

vanitha1822 commented Oct 27, 2026

Copy link
Copy Markdown
Member Author

Rebased to release-3.13.0 and tracked in PR #462

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@vanitha1822 @drtechie @SauravBizbRolly @snehar-nd