Skip to content

Fix public models and add supported vars - #39

Open
fivetran-avinash wants to merge 10 commits into
mainfrom
fix/public-models
Open

Fix public models and add supported vars#39
fivetran-avinash wants to merge 10 commits into
mainfrom
fix/public-models

Conversation

@fivetran-avinash

@fivetran-avinash fivetran-avinash commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

PR Overview

Package version introduced in this PR:

  • 1.3.1

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

  • GA-1031775

Summary of changes:

  • Adding only connector level models not run by app reporting

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 self-assigned this Jul 31, 2026
@fivetran-avinash fivetran-avinash added the docs:ready Triggers the docs generator workflow. label Jul 31, 2026
@fivetran-avinash fivetran-avinash added docs:ready Triggers the docs generator workflow. and removed docs:ready Triggers the docs generator workflow. labels Aug 1, 2026
Comment thread .quickstart/quickstart.yml Outdated
Comment thread .quickstart/quickstart.yml Outdated
Comment thread CHANGELOG.md Outdated
- [`apple_store__subscription_report`](https://fivetran.github.io/dbt_app_reporting/#!/model/model.apple_store.apple_store__subscription_report)
- [`google_play__finance_report`](https://fivetran.github.io/dbt_app_reporting/#!/model/model.google_play.google_play__finance_report)
- Adds support for configuring individual `apple_store` and `google_play` package-level variables (e.g. `apple_store__subscription_events`) directly from Quickstart.
- Adds table variables for non-essential connector source tables (`app_crash_daily`, `stats_store_performance_traffic_source`) so a missing optional source no longer blocks downstream Quickstart models.

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.

This last one should be Under the Hood

Also might wanna specify that these are system-only variables

Comment thread .quickstart/quickstart.yml Outdated
Comment on lines +37 to +40
_fivetran_quickstart_app_crash_daily:
- app_crash_daily
_fivetran_quickstart_stats_store_performance_traffic_source:
- stats_store_performance_traffic_source

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.

Could you include the source name in the variable name so it's easy to tell which is from Apple and which is from Google?

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.

Ah good catch.

@fivetran-avinash fivetran-avinash added docs:ready Triggers the docs generator workflow. and removed docs:ready Triggers the docs generator workflow. labels Aug 4, 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.

2 participants