Skip to content
This repository was archived by the owner on Jan 20, 2026. It is now read-only.

GraphQL support#108

Merged
fivetran-jamie merged 29 commits intomainfrom
update/graphql
Aug 6, 2025
Merged

GraphQL support#108
fivetran-jamie merged 29 commits intomainfrom
update/graphql

Conversation

@fivetran-jamie
Copy link
Contributor

@fivetran-jamie fivetran-jamie commented Jul 24, 2025

PR Overview

Package version introduced in this PR:

  • v0.19.0

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

  • GA-1000203

Summary of changes:

  • Creates GraphQL based staging models and a variable to tell the package which api the connector is on

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 and others added 16 commits July 10, 2025 17:13
* seeds

* Bk

* Bq

* give bigquery what it wants

* vars

* tweak
* seeds

* Bk

* Bq

* give bigquery what it wants

* vars

* tweak

* bk override

* bk vars

* get union data working in bk

* updates

* trailing comma

* seeds

* readme
@fivetran-jamie fivetran-jamie marked this pull request as ready for review July 24, 2025 12:39
Copy link
Contributor

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Choose a reason for hiding this comment

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

@fivetran-jamie great work on all these changes! See below for my comments/questions/suggestions.

Additionally, please address merge conflicts.

Copy link
Contributor

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Choose a reason for hiding this comment

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

LGTM

@fivetran-jamie fivetran-jamie added the docs:ready Triggers the docs generator workflow. label Aug 6, 2025
Co-authored-by: fivetran-catfritz <111930712+fivetran-catfritz@users.noreply.github.com>
@fivetran-jamie fivetran-jamie merged commit 5cbb808 into main Aug 6, 2025
9 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

docs:ready Triggers the docs generator workflow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants