fix: use consistent 'Log in' terminology instead of 'Sign in' by Jayant-Sharma-Dev · Pull Request #16598 · ohcnetwork/care_fe

Jayant-Sharma-Dev · 2026-07-24T17:17:08Z

Proposed Changes

Fixes #16584

Updated the public-facing "Sign in" button to "Log in" for consistent terminology across the authentication flow, matching "Log in as Staff" / "Log in as Patient" already used on the auth/login page.
Added a new log_in translation key in en.json (rather than modifying the existing sign_in key, which is still used elsewhere in the app for other strings).
Also added the log_in key to ml.json (Malayalam) for consistency. Other locale files (hi, ta, mr, kn) were left untouched happy to add those too if needed, wasn't confident enough in the translations to guess.

Before:

After:

Tagging: @ohcnetwork/care-fe-code-reviewers

Merge Checklist

Add specs that demonstrate the bug or test the new feature.
Update product documentation.
Ensure that UI text is placed in I18n files.
Prepare a screenshot or demo video for the changelog entry and attach it to the issue.
Request peer reviews.
Complete QA on mobile devices.
Complete QA on desktop devices.
Add or update Playwright tests for related changes

Summary by CodeRabbit

Bug Fixes
- Updated the logged-out login button to use the correct localized label.
Localization
- Added the English “Log in” translation and aligned the related login text key to improve consistency across languages.

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

coderabbitai · 2026-07-24T17:17:39Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 5535ce9c-c849-43e6-855c-4e9614726e70

📥 Commits

Reviewing files that changed from the base of the PR and between ed0a6d5 and 911255c.

📒 Files selected for processing (3)

public/locale/en.json
public/locale/ml.json
src/components/Common/LoginHeader.tsx

💤 Files with no reviewable changes (1)

public/locale/ml.json

Walkthrough

Changes

Login terminology localization

Layer / File(s)	Summary
Localized login label `public/locale/en.json`, `public/locale/ml.json`, `src/components/Common/LoginHeader.tsx`	The logged-out login button now uses the `log_in` translation key, English defines it as “Log in,” and Malayalam removes the obsolete `sign_in` key.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the main change: replacing "Sign in" with "Log in" for consistency.
Description check	✅ Passed	The description follows the template and includes the required proposed changes, issue reference, tagging, and merge checklist.
Linked Issues check	✅ Passed	The PR updates the public login label to "Log in" and adds matching locale keys, satisfying `#16584`.
Out of Scope Changes check	✅ Passed	The locale edits and button text change are directly tied to the authentication terminology fix.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

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.}

greptile-apps · 2026-07-24T17:18:53Z

Greptile Summary

Updates the public login terminology from “Sign in” to “Log in.”

Adds the English log_in translation key and removes the former English sign_in key.
Updates LoginHeader to use log_in.
Removes the existing sign_in entry from the Malayalam catalog.

Confidence Score: 4/5

The PR should not merge until the unintended direct modification to the Crowdin-managed Malayalam catalog is reverted.

The attempted response to the previous review thread removes an established Malayalam translation rather than leaving the non-English locale file unchanged for Crowdin.

Files Needing Attention: public/locale/ml.json

Important Files Changed

Filename	Overview
public/locale/en.json	Adds the English `log_in` label and removes the superseded `sign_in` key.
public/locale/ml.json	Still directly modifies the Crowdin-managed Malayalam catalog by deleting a pre-existing translation.
src/components/Common/LoginHeader.tsx	Changes the public login button to use the new `log_in` translation key.

_{Reviews (3): Last reviewed commit: "fix: use consistent 'Log in' terminology..." | Re-trigger Greptile}

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

rithviknishad

remove the "sign_in" key from translations after confirming it's not used elsewhere?

Copilot

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

Jayant-Sharma-Dev · 2026-07-25T11:55:52Z

remove the "sign_in" key from translations after confirming it's not used elsewhere?

Checked ...sign_in isn't used anywhere in the codebase anymore .Removed it from both en.json and ml.json.

greptile-apps · 2026-07-25T11:59:30Z

  "show_patient_presets": "രോഗിയുടെ പ്രീസെറ്റുകൾ കാണിക്കുക",
  "show_unread_notifications": "വായിക്കാത്തത് കാണിക്കുക",
-  "sign_in": "സൈൻ ഇൻ",
  "sign_out": "ലോഗ് ഔട്ട് ചെയ്യുക/പുറത്തിറങ്ങുക",


Deletes Crowdin-managed translation

When this branch is merged or synchronized with Crowdin, this deletion removes the pre-existing Malayalam sign_in translation instead of leaving the non-English catalog unchanged, causing the established translation to be lost and the direct edit to conflict with the repository's Crowdin-managed workflow.

Context Used: CLAUDE.md (source)

@rithviknishad Sir so shall I again add sign_in in ml.json?

Jayant-Sharma-Dev · 2026-07-29T08:28:40Z

@NikhilA8606 Any changes ?

nihal467

the page is being fully redesigned, so the issue is not relevant anymore

Jayant-Sharma-Dev requested review from a team and Copilot July 24, 2026 17:17

Copilot AI reviewed Jul 24, 2026

ohc-project-manager Bot assigned Jayant-Sharma-Dev Jul 24, 2026

ohc-project-manager Bot added needs testing needs review labels Jul 24, 2026

greptile-apps Bot reviewed Jul 24, 2026

View reviewed changes

Comment thread public/locale/ml.json Outdated

Copilot AI review requested due to automatic review settings July 24, 2026 17:27

Jayant-Sharma-Dev force-pushed the fix/sign-in-log-in-terminology branch from 9688a68 to ed0a6d5 Compare July 24, 2026 17:27

Copilot AI reviewed Jul 24, 2026

rithviknishad requested changes Jul 25, 2026

View reviewed changes

fix: use consistent 'Log in' terminology instead of 'Sign in'

911255c

Copilot AI review requested due to automatic review settings July 25, 2026 11:53

Jayant-Sharma-Dev force-pushed the fix/sign-in-log-in-terminology branch from ed0a6d5 to 911255c Compare July 25, 2026 11:53

Copilot AI reviewed Jul 25, 2026

github-actions Bot added needs-triage question Further information is requested labels Jul 25, 2026

greptile-apps Bot reviewed Jul 25, 2026

View reviewed changes

nihal467 requested changes Jul 31, 2026

View reviewed changes

nihal467 closed this Jul 31, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

fix: use consistent 'Log in' terminology instead of 'Sign in' - #16598

fix: use consistent 'Log in' terminology instead of 'Sign in'#16598
Jayant-Sharma-Dev wants to merge 1 commit into
ohcnetwork:developfrom
Jayant-Sharma-Dev:fix/sign-in-log-in-terminology

Jayant-Sharma-Dev commented Jul 24, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

coderabbitai Bot commented Jul 24, 2026 •

edited

Loading

Uh oh!

greptile-apps Bot commented Jul 24, 2026 •

edited

Loading

Important Files Changed

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

rithviknishad left a comment

Uh oh!

Copilot AI left a comment

Uh oh!

Jayant-Sharma-Dev commented Jul 25, 2026

Uh oh!

greptile-apps Bot Jul 25, 2026

Uh oh!

Jayant-Sharma-Dev Jul 25, 2026

Uh oh!

Jayant-Sharma-Dev commented Jul 29, 2026

Uh oh!

nihal467 left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

Jayant-Sharma-Dev commented Jul 24, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Proposed Changes

Merge Checklist

Summary by CodeRabbit

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

coderabbitai Bot commented Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

greptile-apps Bot commented Jul 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Greptile Summary

Confidence Score: 4/5

Important Files Changed

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

rithviknishad left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Jayant-Sharma-Dev commented Jul 25, 2026

Uh oh!

greptile-apps Bot Jul 25, 2026

Choose a reason for hiding this comment

Uh oh!

Jayant-Sharma-Dev Jul 25, 2026

Choose a reason for hiding this comment

Uh oh!

Jayant-Sharma-Dev commented Jul 29, 2026

Uh oh!

nihal467 left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Jayant-Sharma-Dev commented Jul 24, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jul 24, 2026 •

edited

Loading

greptile-apps Bot commented Jul 24, 2026 •

edited

Loading