Skip to content

Conversation

@opatrickchen
Copy link

@opatrickchen opatrickchen commented Nov 6, 2025

The app returns ADDITIONAL_CONSENT_REQUIRED when accessing transaction data. This is caused by the Data Transparency Messaging (DTM) enforced by Plaid in US and Canada. This change will get the right consent during sign-up phase.

Summary by CodeRabbit

  • New Features
    • Enhanced financial integration to retrieve transaction data alongside authentication information.

@vercel
Copy link

vercel bot commented Nov 6, 2025

@opatrickchen is attempting to deploy a commit to the JS Mastery Pro Team on Vercel.

A member of the Team first needs to authorize it.

@coderabbitai
Copy link

coderabbitai bot commented Nov 6, 2025

Walkthrough

The Plaid link token request in createLinkToken now includes an additional product 'transactions' alongside the existing 'auth' product. The control flow, error handling, and all other parameters remain unchanged; only the set of requested Plaid products expands.

Changes

Cohort / File(s) Summary
Plaid products expansion
lib/actions/user.actions.ts
Added 'transactions' product to the Plaid link token request alongside 'auth'

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single-line configuration change adding a product string to an existing array
  • No logic modifications, error handling changes, or API contract updates

Poem

🐰 A token grows with transactions in tow,
Alongside auth, the data will flow,
One whisker twitch, a product to show,
Plaid's requests now have more to know! ✨

Pre-merge checks and finishing touches

✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Fix Transactions Display Issue' is directly related to the PR's main objective of obtaining correct consent during sign-up to fix the ADDITIONAL_CONSENT_REQUIRED error when accessing transactions.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b8d5a8e and e7f0bb1.

📒 Files selected for processing (1)
  • lib/actions/user.actions.ts (1 hunks)
🔇 Additional comments (1)
lib/actions/user.actions.ts (1)

141-141: Correct fix for Plaid DTM compliance.

Adding 'transactions' to the products array properly addresses the ADDITIONAL_CONSENT_REQUIRED error by requesting transaction consent during link token creation. This aligns with Plaid's Data Transparency Messaging requirements.

Verification confirms the application actively uses the Plaid transactions product: transactionsSync is called at lib/actions/bank.actions.ts:159, and transaction data is displayed throughout the application (RecentTransactions, TransactionsTable, transaction history page). This is the only location where Plaid products are specified, so no additional changes are required.


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.

1 participant