feat(ui): Add Ko-fi donation entry points - #145
Merged
Conversation
Changes: - Add a public Ko-fi modal from sidebar and footer support controls - Preserve focus when the dialog is dismissed and record future recognition work - Clarify the current repository funding configuration Keeps voluntary support discoverable without adding payment logic or a landing-page prompt.
Changes: - Link the ten existing project skills from the repository root - Keep Claude-Code skill folders as the single source of truth Lets Codex discover the same project workflows without duplicating their instructions.
Changes: - Add the support-system reference and verification guidance - Refresh the README support copy and documentation index Explains the public Ko-fi path, GitHub funding configuration, and consent requirements for future recognition.
❌ Deploy Preview for disscount failed. Why did it fail? →
|
Contributor
Summary by CodeRabbit
WalkthroughAdds a public Ko-fi donation flow with URL-routed modal behavior, navigation entry points, focus restoration, support documentation, funding-message updates, and shared agent-skill symlinks. ChangesDonation support flow
Agent skill links
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant Visitor
participant SupportNavigation
participant ModalRouter
participant DonationModal
participant KoFi
Visitor->>SupportNavigation: Select donation support entry
SupportNavigation->>ModalRouter: Navigate to ?modal=donate
ModalRouter->>DonationModal: Render public modal
DonationModal->>KoFi: Open support page in new tab
DonationModal-->>Visitor: Restore focus after close
Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches 💡 2📝 Generate docstrings 💡
🛠️ Fix failing CI checks 💡
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@frontend/src/constants/navigation.ts`:
- Line 202: Update the comment above the support entry list in navigation
constants to describe it as containing both live support destinations and
coming-soon pages or modals, reflecting the existing donate and contact entries.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: f145a006-1d5d-40de-bfc8-31b85fe4a13c
📒 Files selected for processing (22)
.agents/skills/better-auth-best-practices.agents/skills/create-auth-skill.agents/skills/document-subsystem.agents/skills/email-and-password-best-practices.agents/skills/email-best-practices.agents/skills/frontend-design.agents/skills/multi-tool-code-review.agents/skills/react-email.agents/skills/resend.agents/skills/sentry-nextjs-sdk.github/FUNDING.ymlREADME.mddocs/README.mddocs/SUPPORT.mdfrontend/src/app/(root)/data/landing.tsfrontend/src/components/custom/common/footer-support-icons.tsxfrontend/src/components/custom/donation/donation-modal.tsxfrontend/src/components/custom/modal-router/modal-router.tsxfrontend/src/components/custom/sidebar/sidebar-support-nav.tsxfrontend/src/constants/donation.tsfrontend/src/constants/navigation.tsfrontend/src/lib/modal/modal-registry.ts
| ]; | ||
|
|
||
| // Feedback entry points, shared with the footer; coming-soon until their pages/modals ship. | ||
| // Support entry points, shared with the footer; coming-soon until their pages/modals ship. |
Contributor
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Fix the stale support-list comment.
Line 202 still says support entries are “coming-soon until their pages/modals ship,” but this list now includes the live donate modal and the existing contact modal. Update the comment to describe a mixed live/coming-soon support list.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@frontend/src/constants/navigation.ts` at line 202, Update the comment above
the support entry list in navigation constants to describe it as containing both
live support destinations and coming-soon pages or modals, reflecting the
existing donate and contact entries.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Verification
prettier --writeon changed filesnext typegentsc --noEmiteslint .completed with 27 existing warnings and no errorsMerge order
Merge this before PRs #142, #143, and #144. Those branches must rebase and review the shared modal, navigation, button, or documentation surfaces noted in their follow-up comments.