Skip to content

fix(create): escape apostrophe in placeholder copy#264

Merged
Benjtalkshow merged 1 commit into
boundlessfi:mainfrom
Benjtalkshow:fix/sponsor-role-lint
May 30, 2026
Merged

fix(create): escape apostrophe in placeholder copy#264
Benjtalkshow merged 1 commit into
boundlessfi:mainfrom
Benjtalkshow:fix/sponsor-role-lint

Conversation

@Benjtalkshow
Copy link
Copy Markdown
Contributor

@Benjtalkshow Benjtalkshow commented May 30, 2026

Fixes the react/no-unescaped-entities lint error introduced by #254 on app/bounty/create/page.tsx:47. The placeholder copy had an unescaped ' in "We're" which was failing CI build-and-lint. Replaced with We're.

pnpm lint clean.

Summary by CodeRabbit

  • Style
    • Minor text formatting update on the bounty creation page.

Review Change Stack

Fixes react/no-unescaped-entities lint error introduced in boundlessfi#254.
@vercel
Copy link
Copy Markdown

vercel Bot commented May 30, 2026

@Benjtalkshow is attempting to deploy a commit to the Threadflow Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 30, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 845a4801-18bb-4361-93a8-c7f5bb769f73

📥 Commits

Reviewing files that changed from the base of the PR and between f86c0a6 and 35bf098.

📒 Files selected for processing (1)
  • app/bounty/create/page.tsx

📝 Walkthrough

Walkthrough

The bounty creation page's "Coming Soon" description text is updated to use an HTML-escaped apostrophe (We're) instead of a raw apostrophe. No functional logic or rendering behavior changes.

Changes

Text Content Updates

Layer / File(s) Summary
Coming Soon text HTML escape
app/bounty/create/page.tsx
The "Coming Soon" descriptive text now uses an HTML-escaped apostrophe in "We're" for proper rendering.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

A tiny apostrophe, escaped just right,
In "We're" now glowing with HTML might,
The text now safe, the markup bright,
A rabbit's touch, so small, so tight! 🐰✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely summarizes the main change: escaping an apostrophe in placeholder copy to fix a lint error.
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.

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

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

@Benjtalkshow Benjtalkshow merged commit 9f46a59 into boundlessfi:main May 30, 2026
1 of 4 checks passed
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