Skip to content

Release v1.1.1#29

Closed
fivetran-avinash wants to merge 15 commits into
mainfrom
release/v1.3.1
Closed

Release v1.1.1#29
fivetran-avinash wants to merge 15 commits into
mainfrom
release/v1.3.1

Conversation

@fivetran-avinash

@fivetran-avinash fivetran-avinash commented Jan 7, 2026

Copy link
Copy Markdown
Contributor

PR Overview

Package version introduced in this PR:

  • 1.1.1

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

Summary of changes:

  • Fixes net order amount summation duplication.

Submission Checklist

  • [NA] Alignment meeting with the reviewer (if needed)
    • [NA] Timeline and validation requirements discussed
  • Provide validation details: (see accompanying ticket)
    • 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 changed the title Release/v1.3.1 Release v1.1.1 Jan 7, 2026
@fivetran-avinash fivetran-avinash self-assigned this Jan 7, 2026
@fivetran-avinash fivetran-avinash marked this pull request as ready for review January 7, 2026 15:17
@fivetran-avinash fivetran-avinash added the docs:ready Triggers the docs generator workflow. label Jan 7, 2026
@fivetran-avinash fivetran-avinash added docs:ready Triggers the docs generator workflow. and removed docs:ready Triggers the docs generator workflow. labels Jan 7, 2026

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The code updates look great! I have one CHANGELOG request before approving.

Comment thread CHANGELOG.md

@fivetran-avinash fivetran-avinash left a comment

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.

@fivetran-joemarkiewicz Ready for another look!

Comment thread CHANGELOG.md

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Approved with one minor suggestion in the CHANGELOG to fix a render issue.

Comment thread CHANGELOG.md
- Syntactic fixes applied to consistency tests. ([PR #26](https://github.com/fivetran/dbt_google_play/pull/26))

## Contributors
-[@waligob](https://github.com/waligob) ([PR #28](https://github.com/fivetran/dbt_google_play/pull/28))

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The missing space was causing this to not render properly

Suggested change
-[@waligob](https://github.com/waligob) ([PR #28](https://github.com/fivetran/dbt_google_play/pull/28))
- [@waligob](https://github.com/waligob) ([PR #28](https://github.com/fivetran/dbt_google_play/pull/28))

@fivetran-avinash fivetran-avinash deleted the release/v1.3.1 branch January 7, 2026 20:06
@fivetran-avinash fivetran-avinash added docs:ready Triggers the docs generator workflow. and removed docs:ready Triggers the docs generator workflow. labels Jan 7, 2026
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.

[Bug] dbt Fusion: int_google_play__earnings does not successfully compile against an empty dataset [Bug] Incorrect Summation of Net Order Amount

3 participants