Skip to content

bug/contact-merge-ids#197

Closed
fivetran-catfritz wants to merge 6 commits into
mainfrom
bug/contact-merge-ids
Closed

bug/contact-merge-ids#197
fivetran-catfritz wants to merge 6 commits into
mainfrom
bug/contact-merge-ids

Conversation

@fivetran-catfritz
Copy link
Copy Markdown
Contributor

@fivetran-catfritz fivetran-catfritz commented Mar 9, 2026

PR Overview

Package version introduced in this PR:

  • 1.7.2

This PR addresses the following Issue/Feature(s):

  • GA-1018552

Summary of changes:

  • Changes merged_object_ids data type to a string to prevent merge_contacts macro from throwing an error.

Submission Checklist

  • Alignment meeting with the reviewer (if needed)
    • Timeline and validation requirements discussed
  • Provide validation details:
    • Validation Steps: Check for unintentional effects (e.g., add/run consistency & integrity tests)
    • Testing Instructions: Confirm the change addresses the issue(s)
    • Focus Areas: Complex logic or queries that need extra attention
  • Merge any relevant open PRs into this PR

Changelog

  • Draft changelog for PR
  • Final changelog for release review

@fivetran-avinash fivetran-avinash marked this pull request as ready for review March 27, 2026 16:17
@fivetran-avinash fivetran-avinash self-assigned this Mar 27, 2026
@fivetran-avinash fivetran-avinash added the docs:ready Triggers the docs generator workflow. label Mar 27, 2026
@fivetran-avinash
Copy link
Copy Markdown
Contributor

With @fivetran-catfritz taking on this review, I'll need to close this PR and open up another one off this branch, since she can't review PRs she created.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs:ready Triggers the docs generator workflow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants