Vb/hwc 3.8.1 - #287
Merged
Merged
Conversation
AMM-2127 clinical observation search is not working.
…ronments and fix past-obsteric vanID - All environment files updated to call /getUserFacilityDetails - past-obsteric-history: vanID to facilityID Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
… 104 worklist - inventory.service.ts: vanID to facilityID in redirect URL - lab workarea: techForm vanID to facilityID - 104 referred worklist: vanID to facilityID - environment files: getWorklistByVanID to getWorklistByFacilityID Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…component Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…session fallback Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
…t from session Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add getBenFamilyDetailsUrl to all environment files pointing to identity-api/family/getBenFamilyDetails - Add getBenFamilyDetailsByBenRegId() service method in FamilyTaggingService - Simplify getBeneficiaryDetailsAfterFamilyTag() in FamilyTaggingDetailsComponent to call the new endpoint directly with beneficiaryRegId instead of doing a two-step identityQuickSearch + getFamilyDetails lookup Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Sn/3.6.2
fix: moved latest fix of common ui
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ocation pre-fill Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…facilityID tracing Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- sendToNurseWindow (3 sites): include facilityID in benObject so i_ben_flow_outreach is created with facilityID — without this HWC patients are invisible in nurse worklist - sendBenToAmrit: fix missing JSON.parse on serviceLineDetails and add facilityID to ABHA registration flow Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Fix the Issue in linking Care Context
InventoryService.current_language_set was declared but never assigned, so the guard's error-alert branch threw instead of showing a message whenever authKey/facility/host was missing — masking the real problem and making the redirect look like a no-op. Populate it from HttpServiceService.currentLangugae$, the same pattern AuthGuard already uses. Also guard getppID() against a missing serviceLineDetails entry (JSON.parse(null).parkingPlaceID threw before the redirect could fire), and fix the setLanguage fallback check in worklist.component (sessionStorage.getItem returns null, never undefined).
InventoryService.current_language_set was declared but never assigned, so the guard's error-alert branch threw instead of showing a message whenever authKey/facility/host was missing — masking the real problem and making the redirect look like a no-op. Populate it from HttpServiceService.currentLangugae$, the same pattern AuthGuard already uses. Also guard getppID() against a missing serviceLineDetails entry (JSON.parse(null).parkingPlaceID threw before the redirect could fire), and fix the setLanguage fallback check in worklist.component (sessionStorage.getItem returns null, never undefined).
…ct-3.8.1 fix(pharmacist): stop silent failure when redirecting to Inventory
…sis autocomplete The panel-fill bootstrap logic proactively fetched extra pages whenever the dropdown didn't scroll, flipping noMore to true almost immediately. Now "End of results" only shows when a real fetch (initial search or scroll-triggered page) actually returns an empty response. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sn/fix pharmacist inventory redirect
…-context popup (#286) After a beneficiary returns from the lab and the doctor clicks Update, getHealthIDDetails() concatenated res.data.response directly into the popup text. When that field was falsy, `undefined + '. '` rendered the literal word "undefined" in the success dialog. Fall back to a proper success string, matching the pattern already used by the initial Save flow. Co-authored-by: Sneha <sneha@ADMINs-MacBook-Pro.local> Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Contributor
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
Resolves the package-lock.json conflict in PR #287. package.json merged cleanly on its own, keeping each side's intended change: - @angular/material-moment-adapter ^16.2.12 and ngx-bootstrap ^11.0.0 (vb/hwc-3.8.1's deliberate downgrades for Angular 16 compatibility) - jasmine-core ~5.12.0 (release-3.9.0's dependabot bump) package-lock.json was regenerated from the merged package.json with `npm install --package-lock-only --lockfile-version 2` rather than hand-resolved, so the tree is internally consistent. lockfileVersion 2 is preserved to match both sides; resolved versions are jasmine-core 5.12.1, material-moment-adapter 16.2.14, ngx-bootstrap 11.0.2. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
vanitha1822
approved these changes
Aug 27, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Description
JIRA ID:
Please provide a summary of the change and the motivation behind it. Include relevant context and details.
✅ Type of Change
ℹ️ Additional Information
Please describe how the changes were tested, and include any relevant screenshots, logs, or other information that provides additional context.