Skip to content

Fix lint failure caused by unescaped entities in create page#224

Open
Abhist17 wants to merge 1 commit intoAOSSIE-Org:mainfrom
Abhist17:feat/first-contribution
Open

Fix lint failure caused by unescaped entities in create page#224
Abhist17 wants to merge 1 commit intoAOSSIE-Org:mainfrom
Abhist17:feat/first-contribution

Conversation

@Abhist17
Copy link

@Abhist17 Abhist17 commented Jan 27, 2026

Fixes #220

This PR resolves a blocking ESLint error caused by unescaped quotes
in app/create/page.tsx. After this change, npm run lint completes
successfully without build-breaking errors.

Summary by CodeRabbit

  • Bug Fixes
    • Fixed quotation mark rendering in the "No candidates added yet..." message on the Create page to display correctly.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 27, 2026

📝 Walkthrough

Walkthrough

A message string in the Create page was updated to use HTML entity " instead of literal double quotes to comply with ESLint's react/no-unescaped-entities rule.

Changes

Cohort / File(s) Summary
Quote Entity Fix
client/app/create/page.tsx
Updated unescaped double quotes in the "No candidates added yet..." message to use HTML entity " to satisfy ESLint react/no-unescaped-entities rule

Estimated Code Review Effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A quote was lost in the linting game,
Unescaped and wild, it brought us shame.
But " saved the day, so neat and small,
The lint now passes—no errors at all! 🐰✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly matches the PR objective: fixing a lint failure caused by unescaped entities in the create page, which aligns with the actual code change converting double quotes to HTML entities.
Linked Issues check ✅ Passed The PR successfully addresses issue #220 by replacing unescaped double quotes with " HTML entity, ensuring npm run lint passes without the react/no-unescaped-entities error.
Out of Scope Changes check ✅ Passed The change is narrowly focused on fixing the reported lint error in client/app/create/page.tsx by replacing unescaped quotes, with no extraneous modifications to other files or unrelated functionality.
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

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.

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.

BUG: Lint command fails due to unescaped entities in app/create/page.tsx

1 participant