Skip to content

fix(ad_insights): exclude marketing_messages_website_purchase_values field - #448

Merged
edgarrmondragon merged 1 commit into
MeltanoLabs:mainfrom
ebnermic:bugfix/exclude-marketing-messages-website-purchase-values
Jul 30, 2026
Merged

fix(ad_insights): exclude marketing_messages_website_purchase_values field#448
edgarrmondragon merged 1 commit into
MeltanoLabs:mainfrom
ebnermic:bugfix/exclude-marketing-messages-website-purchase-values

Conversation

@ebnermic

Copy link
Copy Markdown
Contributor

Summary

  • Adds marketing_messages_website_purchase_values to EXCLUDED_FIELDS in ad_insights.py
  • Field is rejected by the Facebook Ads Insights API with error code #100: marketing_messages_website_purchase_values is not valid for fields param
  • Follows the same pattern as other already-excluded marketing_messages_* fields

Test plan

  • Run tap-facebook with an account that previously triggered the error
  • Confirm sync completes without #100 API error on ad_insights stream

🤖 Generated with Claude Code

@ebnermic

Copy link
Copy Markdown
Contributor Author

Hi @edgarrmondragon,

Would it be possible to get a review on this?
Our pipeline is currently failing due to this, otherwise we would need to work with the fork which would be a bit of a shame.

Thank a lot!

…field

Field is not valid for the Facebook Ads Insights API fields param,
causing sync failures with error code MeltanoLabs#100.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Edgar Ramírez Mondragón <edgarrm358@gmail.com>
@edgarrmondragon
edgarrmondragon force-pushed the bugfix/exclude-marketing-messages-website-purchase-values branch from 52756cb to 7908e07 Compare July 30, 2026 15:44
@edgarrmondragon
edgarrmondragon merged commit 43f0068 into MeltanoLabs:main Jul 30, 2026
2 of 6 checks passed
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.

2 participants