Skip to content

feature/update_tiktok_ads_version#176

Merged
fivetran-savage merged 4 commits into
mainfrom
feature/update_tiktok_ads_version
May 18, 2026
Merged

feature/update_tiktok_ads_version#176
fivetran-savage merged 4 commits into
mainfrom
feature/update_tiktok_ads_version

Conversation

@fivetran-savage

Copy link
Copy Markdown
Contributor

PR Overview

Package version introduced in this PR:

  • 2.6.0

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

Summary of changes:

  • Updates required tiktok_ads version to 1.3.0

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-jamie fivetran-jamie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks awesome, just some changelog comments!

Comment thread CHANGELOG.md
Comment thread CHANGELOG.md Outdated
Comment on lines +5 to +6
## Upstream Dependency Change
- Increases the required TikTok Ads package to version [1.3.0](https://github.com/fivetran/dbt_tiktok_ads/blob/main/CHANGELOG.md#dbt_tiktok_ads-v130)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Not sure we need to include this in the changelog -- whatcha think?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Don't dbt users need to know that they need to update their tiktok_ads version or is that automatic?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Automatic since ad_reporting itself installs tiktok_ads!

@fivetran-jamie fivetran-jamie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Approved with one edit to the changelog!

Comment thread CHANGELOG.md Outdated
Co-authored-by: Jamie Rodriguez <65564846+fivetran-jamie@users.noreply.github.com>
@fivetran-savage fivetran-savage added the docs:ready Triggers the docs generator workflow. label May 18, 2026
@fivetran-savage fivetran-savage merged commit 0e0beb8 into main May 18, 2026
8 checks passed
@fivetran-savage fivetran-savage deleted the feature/update_tiktok_ads_version branch May 18, 2026 18:35
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