Skip to content

fix: call report issue for 104 - #445

Merged
vishwab1 merged 6 commits into
release-3.9.0from
nd/vs/AMM-2195
Aug 19, 2026
Merged

fix: call report issue for 104#445
vishwab1 merged 6 commits into
release-3.9.0from
nd/vs/AMM-2195

Conversation

@vanitha1822

@vanitha1822 vanitha1822 commented Aug 4, 2026

Copy link
Copy Markdown
Member

📋 Description

JIRA ID:
AMM-2195

✅ Type of Change

  • 🐞 Bug fix (non-breaking change which resolves an issue)

@coderabbitai

coderabbitai Bot commented Aug 4, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: fc34afa6-2c3c-48d5-96b4-fc545a45fb83

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@sonarqubecloud

sonarqubecloud Bot commented Aug 7, 2026

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
10 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

@vishwab1
vishwab1 merged commit 99b17d8 into release-3.9.0 Aug 19, 2026
5 checks passed
vishwab1 pushed a commit that referenced this pull request Sep 22, 2026
@vanitha1822 @SauravBizbRolly
@snehar-nd Sneha @claude
Merge Release 3.9.0 with Main (#456)
f5f68dd 
* fix: facility id null (#433)

* fix order issue in dynamic from

* fix: call report issue for 104 (#445)

* 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

* fix: use valid HTTP status 500 instead of invalid 5000 in report error 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>

* Revert the unwanted scheduler changes (#449)

* Revert "fix: add the range to sync"

This reverts commit 92b3f20.

* Revert "fix: call report issue for 104"

This reverts commit a47b709.

* fix: remove the wrong column (#454)

* fix: build issue (#455)

* fix order issue in dynamic from (#447) (#451)

---------

Co-authored-by: Saurav Mishra <saurav.mishra@bizbrolly.com>
Co-authored-by: SnehaRH <77656297+snehar-nd@users.noreply.github.com>
Co-authored-by: Sneha <sneha@ADMINs-MacBook-Pro.local>
Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Co-authored-by: Saurav Mishra <80103738+SauravBizbRolly@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vishwab1 vishwab1 vishwab1 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@vanitha1822 @vishwab1