Skip to content

fix(organizations): correct casing in Git token empty state copy - #2868

Closed
RemiBonnet wants to merge 1 commit into
stagingfrom
fix/git-token-empty-state-copy
Closed

fix(organizations): correct casing in Git token empty state copy#2868
RemiBonnet wants to merge 1 commit into
stagingfrom
fix/git-token-empty-state-copy

Conversation

@RemiBonnet

Copy link
Copy Markdown
Member

Summary

Issue: trivial copy fix

Corrects the casing of the Git token list empty-state message so "Tokens" follows sentence case, matching the surrounding UI copy.

  • libs/domains/organizations/feature/src/lib/git-token-list/git-token-list.tsx: change "No Git Tokens found." to "No Git tokens found." in the empty state.

Screenshots / Recordings

N/A — text-only change.

Testing

  • Changes tested locally in the relevant Console's pages and Storybooks
  • yarn test or yarn test -u (if you need to regenerate snapshots)
  • yarn format
  • yarn lint

Note: the Node/Yarn toolchain is not available in the agent environment, so yarn format/yarn test/yarn lint could not be executed. This is a single-word copy change in a JSX string literal; it does not affect the existing git-token-list snapshot (which renders the populated list, not the empty state).

PR Checklist

  • I followed naming, styling, and TypeScript rules (see .cursor/rules)
  • I performed a self-review (diff inspected, dead code removed)
  • I titled the PR using Conventional Commits with a scope when possible
  • I only kept necessary comments, written in English (watch for useless AI comments)
  • I involved a designer to validate UI changes if I am not a designer
  • I covered new business logic with tests (unit)
  • I confirmed CI is green (Codecov red can be accepted)
  • I reviewed and executed locally any AI-assisted code

@nx-cloud

nx-cloud Bot commented Aug 13, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 86b44f2

Command Status Duration Result
nx run console:build --parallel=3 --configurati... ✅ Succeeded 1m 3s View ↗
nx affected --target=test --parallel=3 --config... ✅ Succeeded 3m 1s View ↗
nx affected --target=lint --parallel=3 ✅ Succeeded 2m 49s View ↗
nx-cloud record -- yarn nx format:check ✅ Succeeded 6s View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗


☁️ Nx Cloud last updated this comment at 2026-08-13 12:09:22 UTC

@codecov

codecov Bot commented Aug 13, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 46.39%. Comparing base (1435ca6) to head (86b44f2).

Additional details and impacted files
@@             Coverage Diff             @@
##           staging    #2868      +/-   ##
===========================================
- Coverage    49.58%   46.39%   -3.20%     
===========================================
  Files          850      662     -188     
  Lines        21162    16188    -4974     
  Branches      6375     4803    -1572     
===========================================
- Hits         10494     7511    -2983     
+ Misses        8898     7412    -1486     
+ Partials      1770     1265     -505     
Flag Coverage Δ
unittests 46.39% <ø> (-3.20%) ⬇️

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.

@RemiBonnet RemiBonnet closed this Aug 13, 2026
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