Skip to content

Add missing space under Controls > Setup experience > Users > End user authentication#49074

Open
spalmesano0 wants to merge 1 commit into
mainfrom
missing-space
Open

Add missing space under Controls > Setup experience > Users > End user authentication#49074
spalmesano0 wants to merge 1 commit into
mainfrom
missing-space

Conversation

@spalmesano0

@spalmesano0 spalmesano0 commented Jul 9, 2026

Copy link
Copy Markdown
Member

We missed a space under Controls > Setup experience > Users > End user authentication.

Before:

Screenshot 2026-07-09 at 14 56 09

After:

Screenshot 2026-07-09 at 14 56 04

Checklist for submitter

Testing

  • QA'd all new/changed functionality manually

Summary by CodeRabbit

  • Bug Fixes
    • Improved the spacing in the “Lock end user info” help text so the highlighted field names now display more clearly in the UI.

Copilot AI review requested due to automatic review settings July 9, 2026 19:58
@spalmesano0 spalmesano0 requested a review from a team as a code owner July 9, 2026 19:58
@spalmesano0 spalmesano0 changed the title Add missing space Add missing space under Controls > Setup experience > Users > End user authentication Jul 9, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR fixes a UI text spacing issue in the “End user authentication” section under Controls > Setup experience > Users, ensuring proper whitespace between the emphasized field names and the following sentence text.

Changes:

  • Adds an explicit JSX whitespace token ({" "}) after <strong>Full name</strong> in the help text to ensure the rendered sentence has the correct spacing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e8bc24ab-f177-4e24-95b9-4370d8bb274d

📥 Commits

Reviewing files that changed from the base of the PR and between 318c85f and 6d5281b.

📒 Files selected for processing (1)
  • frontend/pages/ManageControlsPage/SetupExperience/cards/Users/components/UsersForm/components/EndUserAuthSection/EndUserAuthSection.tsx

Walkthrough

This change makes a single-line edit to the help text rendered under the "Lock end user info" checkbox in EndUserAuthSection.tsx, adding an explicit JSX space between the "Account Name" and "Full name" bolded text segments to correct the spacing displayed in the UI.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description lacks the required Related issue line and most template checklist sections, so it is largely incomplete. Add the Related issue field and fill or remove the applicable checklist items, including testing and any other required sections from the template.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the UI spacing fix in Controls > Setup experience > Users > End user authentication.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch missing-space

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.

@codecov

codecov Bot commented Jul 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 68.12%. Comparing base (598f425) to head (6d5281b).
⚠️ Report is 16 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #49074   +/-   ##
=======================================
  Coverage   68.11%   68.12%           
=======================================
  Files        3731     3731           
  Lines      235434   235447   +13     
  Branches    12520    12528    +8     
=======================================
+ Hits       160375   160389   +14     
+ Misses      60669    60668    -1     
  Partials    14390    14390           
Flag Coverage Δ
frontend 59.31% <ø> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants