Skip to content

fix: Fix missing tags in Conversations stream#92

Merged
fcortevargas merged 15 commits intomainfrom
fernando/fix-missing-tags-in-conversations-stream
Mar 24, 2026
Merged

fix: Fix missing tags in Conversations stream#92
fcortevargas merged 15 commits intomainfrom
fernando/fix-missing-tags-in-conversations-stream

Conversation

@fcortevargas
Copy link
Copy Markdown
Contributor

No description provided.

@fcortevargas fcortevargas requested a review from a team as a code owner March 11, 2026 12:38
Copilot AI review requested due to automatic review settings March 11, 2026 12:38
@fcortevargas fcortevargas marked this pull request as draft March 11, 2026 12:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR targets diagnosing/fixing missing tag data in the Intercom Conversations stream by adding record-level inspection in post_process.

Changes:

  • Added a post_process override in ConversationsStream to log conversations whose tag names contain "aircall".
  • Introduced a structured log line containing conversation id and matching tags.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread tap_intercom/streams.py Outdated
Comment thread tap_intercom/streams.py Outdated
Comment thread tap_intercom/streams.py Outdated
@fcortevargas fcortevargas force-pushed the fernando/fix-missing-tags-in-conversations-stream branch from 3154d42 to bc365a5 Compare March 11, 2026 13:03
@fcortevargas fcortevargas force-pushed the fernando/fix-missing-tags-in-conversations-stream branch from c1dbc1d to ca44444 Compare March 17, 2026 14:19
@fcortevargas fcortevargas marked this pull request as ready for review March 24, 2026 12:20
@fcortevargas fcortevargas merged commit 15c0dc7 into main Mar 24, 2026
8 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