Repository navigation
[v4] Feat: Add IRA payment method and new DAF Icon - #471
Merged
johnemau merged 1 commit intoOct 8, 2026
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
johnemau
changed the title
johnemau
changed the title
johnemau
marked this pull request as ready for review
October 8, 2026 19:04
johnemau
requested review from
Exploding,
Fishbakh-N,
jiwon85 and
santiher
and
a balanced review from Copilot
October 8, 2026 19:04
johnemau
added this pull request to stack #472
October 8, 2026 19:05
There was a problem hiding this comment.
🟡 Changes recommended
IRA currently displays incorrect tax-deductibility language in the FAQ and footer.
1 open finding
What changed in this PR
Adds IRA donations to the v4 widget and refreshes the DAF payment icon.
Changes:
- Adds IRA selection, ordering, one-time enforcement, fee copy, and icon.
- Replaces the branded DAFPay icon with a configurable-color DAF icon.
| File | Description |
|---|---|
PaymentMethod.ts |
Registers and orders IRA as a one-time method. |
IraIcon.tsx |
Adds the IRA icon. |
DafIcon.tsx |
Replaces the DAF icon. |
NameForPaymentMethod.tsx |
Adds the IRA label. |
IconForPaymentMethod.tsx |
Maps IRA to its icon. |
Frequency/index.tsx |
Adds IRA one-time messaging. |
Faq/helpers.tsx |
Adds IRA fee information. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
jiwon85
approved these changes
Oct 8, 2026
Problem: Embeds such as projectceti.org do not offer IRA, although every.org's donate page does. Root Cause: The widget's PaymentMethod enum has no `ira` value. mergeConfig and parseDonateUrl drop methods outside the enum, and the default method list is built from the enum. Solution: Add `ira` with its name and icon, list it as one-time only like DAF, and add the every.org fee FAQ text. Submitting redirects to every.org with `method=ira`, which opens the existing IRA (QCD) flow. Replace the DAF icon (Chariot "DAF pay" logo) with every.org's outline DAF icon so it matches the new IRA icon and every.org's donate page. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
johnemau
force-pushed
the
john-with-claude/ira-donate-page-missing-0eb17f
branch
from
October 8, 2026 19:33
048e61e to
f8259b1
Compare
This branch was successfully deployed
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
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.
Parody with Every.org
Stacked on
🤖 Generated with Claude Code