Repository navigation
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
johnemau
force-pushed
the
john-with-claude/widget-tax-statements-stacked
branch
from
October 8, 2026 21:30
cf5445b to
c925bd2
Compare
johnemau
changed the title
johnemau
marked this pull request as ready for review
October 8, 2026 21:44
johnemau
enabled auto-merge (squash)
October 8, 2026 21:45
There was a problem hiding this comment.
🟡 Changes recommended
The FAQ introduction still incorrectly promises tax-deduction eligibility for DAF and IRA donations.
1 open finding
What changed in this PR
Aligns DAF and IRA receipt and tax messaging with Every.org.
Changes:
- Adds payment-method-specific DAF and IRA statements.
- Updates the FAQ and footer based on the selected payment method.
| File | Description |
|---|---|
Footer/index.tsx |
Passes the selected payment method to footer messaging. |
Footer/helpers.tsx |
Adds DAF and IRA tax statements. |
Faq/index.tsx |
Provides method-specific receipt answers. |
🧠 Review effort: Balanced
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Problem: With DAF or IRA selected, the widget's footer and tax FAQ say "100% of your donation is tax-deductible", although neither receipt is tax-deductible. The intro FAQ also says Every.org immediately sends a receipt, which it does not do for pledge-based methods such as DAF and IRA. Root Cause: The footer and FAQ never pass the selected method to getTaxDeductibleStatement, so DAF_DEDUCTIBLE_STATEMENT is never shown, and there is no IRA (QCD) statement. Solution: Add every.org's IRA_DEDUCTIBLE_STATEMENT and pass the selected method in the footer. Replace the "Is this donation tax-deductible?" and "Will I receive a receipt for my donation?" FAQ items with every.org's single "Will I receive a tax-deductible receipt for my donation?" item, which shows only the DAF or IRA statement for those methods. Say the receipt is sent once a gift has completed processing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
johnemau
force-pushed
the
john-with-claude/widget-tax-statements-stacked
branch
from
October 8, 2026 22:05
c925bd2 to
904db43
Compare
johnemau
changed the title
johnemau
requested review from
Exploding,
Fishbakh-N,
csday,
jiwon85 and
santiher
October 8, 2026 22:54
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.
Also See:
main)🤖 Generated with Claude Code