Skip to content

Changed Advance search to Advanced search#122

Closed
snehar-nd wants to merge 2 commits intorelease-3.6.2from
sn/es
Closed

Changed Advance search to Advanced search#122
snehar-nd wants to merge 2 commits intorelease-3.6.2from
sn/es

Conversation

@snehar-nd
Copy link
Contributor

@snehar-nd snehar-nd commented Feb 13, 2026

📋 Description

JIRA ID:

AMM-2159

✅ Type of Change

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

ℹ️ Additional Information

Changed text to Advanced Search.

Summary by CodeRabbit

  • Documentation

    • Corrected spelling in search interface label from "Advance Search" to "Advanced Search."
  • Chores

    • Updated environment configuration across all deployment environments.

@coderabbitai
Copy link

coderabbitai bot commented Feb 13, 2026

📝 Walkthrough

Walkthrough

Configuration and localization updates across the codebase: spelling correction for "Advance Search" to "Advanced Search" in language assets, and addition of a new environment configuration flag isEnableES (set to false) across all environment configuration files.

Changes

Cohort / File(s) Summary
Language & Asset Files
src/assets/English.json, src/assets/language.json
Corrected spelling: "Advance Search" → "Advanced Search" in bendetails section.
Environment Configuration
src/environments/environment.*.ts*
Added new boolean configuration property isEnableES: false across all environment files (ci.ts.template, dev.ts, development.ts, local.ts, prod.ts, test.ts).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~4 minutes

Poem

🐰 A whisker-twitch of spelling grace,
"Advanced" takes the proper place,
Across the config lands so wide,
A flag called isEnableES glides,
Small tweaks with bunny-hop delight! ✨

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Merge Conflict Detection ⚠️ Warning ❌ Merge conflicts detected (19 files):

⚔️ Common-UI (content)
⚔️ pom.xml (content)
⚔️ src/app/app-modules/core/services/confirmation.service.ts (content)
⚔️ src/app/app-modules/core/services/http-interceptor.service.ts (content)
⚔️ src/app/app-modules/nurse-doctor/anc/anc.component.ts (content)
⚔️ src/app/app-modules/nurse-doctor/history/general-opd-history/personal-history/personal-history.component.ts (content)
⚔️ src/app/app-modules/nurse-doctor/nurse-worklist-wrapper/nurse-mmu-tm-referred-worklist/nurse-mmu-tm-referred-worklist.component.ts (content)
⚔️ src/app/app-modules/nurse-doctor/pnc/pnc.component.ts (content)
⚔️ src/app/app-modules/nurse-doctor/shared/services/nurse.service.ts (content)
⚔️ src/assets/Assamese.json (content)
⚔️ src/assets/English.json (content)
⚔️ src/assets/Hindi.json (content)
⚔️ src/assets/language.json (content)
⚔️ src/environments/environment.ci.ts.template (content)
⚔️ src/environments/environment.dev.ts (content)
⚔️ src/environments/environment.development.ts (content)
⚔️ src/environments/environment.local.ts (content)
⚔️ src/environments/environment.prod.ts (content)
⚔️ src/environments/environment.test.ts (content)

These conflicts must be resolved before merging into release-3.6.2.
Resolve conflicts locally and push changes to this branch.
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title correctly summarizes the main text change from 'Advance Search' to 'Advanced Search' across multiple JSON files, which is the primary user-facing change in this PR.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch sn/es
⚔️ Resolve merge conflicts (beta)
  • Auto-commit resolved conflicts to branch sn/es
  • Create stacked PR with resolved conflicts
  • Post resolved changes as copyable diffs in a comment

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 and usage tips.

@sonarqubecloud
Copy link

@snehar-nd snehar-nd changed the base branch from release-3.6.1 to release-3.6.2 February 16, 2026 10:48
@snehar-nd snehar-nd changed the title Sn/es Changed Advance search to Advanced search Feb 16, 2026
@snehar-nd
Copy link
Contributor Author

review PR @CodeRabbit

@coderabbitai
Copy link

coderabbitai bot commented Feb 16, 2026

@snehar-nd: I'll review the PR changes for you.

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/assets/English.json (1)

301-301: ⚠️ Potential issue | 🟡 Minor

Missed occurrence: "Advance Beneficiary Search" should also be "Advanced Beneficiary Search".

This is the same typo the PR aims to fix. Update this value for consistency.

🔧 Suggested fix
-            "advanceBeneficiarySearch": "Advance Beneficiary Search",
+            "advanceBeneficiarySearch": "Advanced Beneficiary Search",
🧹 Nitpick comments (2)
src/environments/environment.dev.ts (1)

520-521: Inconsistent indentation: 4 spaces vs. 2 spaces used by the rest of the object.

The new property uses 4-space indentation while all other properties in the environment object use 2-space indentation.

🔧 Suggested fix
-    isEnableES: false,
-
+  isEnableES: false,
src/environments/environment.prod.ts (1)

514-515: Inconsistent indentation and trailing blank line.

Line 514 uses 4-space indentation while every other property in this object uses 2 spaces. There's also a stray blank line (515) before the closing };. This is minor but worth keeping consistent with the rest of the file.

🧹 Suggested fix
-    isEnableES: false,
-
+  isEnableES: false,
 };

@snehar-nd snehar-nd closed this Feb 18, 2026
@snehar-nd snehar-nd deleted the sn/es branch February 18, 2026 08:52
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.

1 participant