From a492d4573331be75a0ad55e7ea3b8ac1fbec4611 Mon Sep 17 00:00:00 2001 From: Mahdi Dibaiee Date: Fri, 8 Nov 2024 14:35:13 +0000 Subject: [PATCH] dbt: revert required fields, the UI now allow removing the object See https://github.com/estuary/ui/issues/1353 --- go/dbt/trigger.go | 8 ++++---- materialize-bigquery/.snapshots/TestSpecification | 5 +++++ materialize-databricks/.snapshots/TestSpecification | 5 +++++ materialize-mysql/.snapshots/TestSpecification | 5 +++++ materialize-postgres/.snapshots/TestSpecification | 5 +++++ materialize-redshift/.snapshots/TestSpecification | 5 +++++ materialize-snowflake/.snapshots/TestSpecification | 5 +++++ materialize-sqlserver/.snapshots/TestSpecification | 5 +++++ 8 files changed, 39 insertions(+), 4 deletions(-) diff --git a/go/dbt/trigger.go b/go/dbt/trigger.go index 800b40d0e..3ed015917 100644 --- a/go/dbt/trigger.go +++ b/go/dbt/trigger.go @@ -12,11 +12,11 @@ import ( ) type JobConfig struct { - JobID string `json:"job_id,omitempty" jsonschema:"title=Job ID,description=dbt job ID"` - AccountID string `json:"account_id,omitempty" jsonschema:"title=Account ID,description=dbt account ID"` + JobID string `json:"job_id" jsonschema:"title=Job ID,description=dbt job ID"` + AccountID string `json:"account_id" jsonschema:"title=Account ID,description=dbt account ID"` AccessURL string `json:"access_url,omitempty" jsonschema:"title=Access URL,description=dbt access URL can be found in your Account Settings. See go.estuary.dev/dbt-cloud-trigger" jsonschema_extras:"pattern=^https://.+$"` - AccountPrefix string `json:"account_prefix,omitempty" jsonschema:"-"` - APIKey string `json:"api_key,omitempty" jsonschema:"title=API Key,description=dbt API Key" jsonschema_extras:"secret=true"` + AccountPrefix string `json:"account_prefix" jsonschema:"-"` + APIKey string `json:"api_key" jsonschema:"title=API Key,description=dbt API Key" jsonschema_extras:"secret=true"` Cause string `json:"cause,omitempty" jsonschema:"title=Cause Message,description=You can set a custom 'cause' message for the job trigger. Defaults to 'Estuary Flow'."` Mode string `json:"mode,omitempty" jsonschema:"title=Job Trigger Mode,description=Specifies how should already-running jobs be treated. Defaults to 'skip' which skips the trigger if a job is already running; 'replace' cancels the running job and runs a new one; while 'ignore' triggers a new job regardless of existing jobs.,enum=skip,enum=replace,enum=ignore,default=skip"` Interval string `json:"interval,omitempty" jsonschema:"title=Minimum Run Interval,description=Minimum time between dbt job triggers. This interval is only triggered if data has been materialized by your task.,default=30m"` diff --git a/materialize-bigquery/.snapshots/TestSpecification b/materialize-bigquery/.snapshots/TestSpecification index 7d19ad88d..9ae854276 100644 --- a/materialize-bigquery/.snapshots/TestSpecification +++ b/materialize-bigquery/.snapshots/TestSpecification @@ -153,6 +153,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt job when new data is available" } diff --git a/materialize-databricks/.snapshots/TestSpecification b/materialize-databricks/.snapshots/TestSpecification index 330036c3b..5e37b18fc 100644 --- a/materialize-databricks/.snapshots/TestSpecification +++ b/materialize-databricks/.snapshots/TestSpecification @@ -167,6 +167,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt Job when new data is available" } diff --git a/materialize-mysql/.snapshots/TestSpecification b/materialize-mysql/.snapshots/TestSpecification index e8e748856..78c50b5f1 100644 --- a/materialize-mysql/.snapshots/TestSpecification +++ b/materialize-mysql/.snapshots/TestSpecification @@ -90,6 +90,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt Job when new data is available" }, diff --git a/materialize-postgres/.snapshots/TestSpecification b/materialize-postgres/.snapshots/TestSpecification index 8203956af..0eeb6cf56 100644 --- a/materialize-postgres/.snapshots/TestSpecification +++ b/materialize-postgres/.snapshots/TestSpecification @@ -91,6 +91,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt Job when new data is available" }, diff --git a/materialize-redshift/.snapshots/TestSpecification b/materialize-redshift/.snapshots/TestSpecification index 70932cb9d..9ff65644c 100644 --- a/materialize-redshift/.snapshots/TestSpecification +++ b/materialize-redshift/.snapshots/TestSpecification @@ -172,6 +172,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt Job when new data is available" }, diff --git a/materialize-snowflake/.snapshots/TestSpecification b/materialize-snowflake/.snapshots/TestSpecification index 34c8d5f11..c9b28a491 100644 --- a/materialize-snowflake/.snapshots/TestSpecification +++ b/materialize-snowflake/.snapshots/TestSpecification @@ -214,6 +214,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt Job when new data is available" } diff --git a/materialize-sqlserver/.snapshots/TestSpecification b/materialize-sqlserver/.snapshots/TestSpecification index 97c310d9e..08eb6f059 100644 --- a/materialize-sqlserver/.snapshots/TestSpecification +++ b/materialize-sqlserver/.snapshots/TestSpecification @@ -90,6 +90,11 @@ }, "additionalProperties": false, "type": "object", + "required": [ + "job_id", + "account_id", + "api_key" + ], "title": "dbt Cloud Job Trigger", "description": "Trigger a dbt Job when new data is available" },