feat(fb_custom_audience): add accounts framework support#2363
feat(fb_custom_audience): add accounts framework support#2363shekhar-rudder wants to merge 1 commit intodevelopfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (8)
📝 WalkthroughWalkthroughThe changes introduce a new account-based authentication method for the Facebook Custom Audience destination. A new access token account type configuration is added alongside modifications to the parent destination configuration to support selecting pre-configured accounts via rudderAccountId, enforcing mutual exclusivity between manual accessToken and managed accounts through schema constraints. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/configurations/destinations/fb_custom_audience/schema.json`:
- Around line 6-8: The schema field rudderAccountId currently allows an empty
string; update the JSON schema for rudderAccountId to require a non-empty value
(for example add "minLength": 1 or a non-empty pattern) so the credential oneOf
branch cannot be satisfied by "" at runtime; locate the rudderAccountId
definition in src/configurations/destinations/fb_custom_audience/schema.json and
add the non-empty constraint (minLength or pattern) to enforce a usable
credential.
ℹ️ Review info
Configuration used: defaults
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
src/configurations/destinations/fb_custom_audience/accounts/fb_custom_audience_access_token/db-config.jsonsrc/configurations/destinations/fb_custom_audience/accounts/fb_custom_audience_access_token/schema.jsonsrc/configurations/destinations/fb_custom_audience/accounts/fb_custom_audience_access_token/ui-config.jsonsrc/configurations/destinations/fb_custom_audience/db-config.jsonsrc/configurations/destinations/fb_custom_audience/schema.jsonsrc/configurations/destinations/fb_custom_audience/ui-config.jsontest/data/validation/destinations/fb_custom_audience.json
Adds Accounts Framework support for Facebook Custom Audience destination as part of INT-5928. Introduces a new access token account definition and updates destination configs to support rudderAccountId alongside the existing accessToken field for a backward-compatible migration. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
789673c to
a7515ad
Compare
...rations/destinations/fb_custom_audience/accounts/fb_custom_audience_access_token/schema.json
Show resolved
Hide resolved
| @@ -3,6 +3,9 @@ | |||
| "displayName": "Facebook Custom Audience", | |||
| "config": { | |||
| "features": ["vdm-next"], | |||
There was a problem hiding this comment.
does this "features": has any usage ?
There was a problem hiding this comment.
I don't see any references in the webapp. @arpl Can you confirm?
Summary
Adds Accounts Framework support for the Facebook Custom Audience destination as part of INT-5928.
fb_custom_audience_access_token): introducesdb-config.json,ui-config.json, andschema.jsonunderaccounts/for theDESTINATION_FB_CUSTOM_AUDIENCE_ACCESS_TOKENaccount type, supportingaccess_tokenauthentication withaccessTokenas a secret field.db-config.json: registerssupportedAccountDefinitionsmappingrudderAccountIdto the new account definition, and addsrudderAccountIdtodestConfig.defaultConfig.ui-config.json: adds anaccountManagementInputfield (rudderAccountId) in the Connection Settings group alongside the existingaccessTokenfield for backward compatibility.schema.json: addsrudderAccountIdas a string property and introduces aoneOfconstraint ensuring exactly one ofaccessTokenorrudderAccountIdis provided — not both, not neither.rudderAccountId-only path, the invalid both-present case, and the invalid neither-present case.Summary by CodeRabbit
Release Notes
New Features
Tests