-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bugfix/daylight-end-catch #128
Changes from all commits
691cec5
03524ea
ec2d637
93a5575
9baabce
163a01d
fd89c07
0384705
f9b3f64
550df6f
a9d2b1b
aae287d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,13 @@ | ||
name: 'auto release' | ||
on: | ||
pull_request: | ||
types: | ||
- closed | ||
branches: | ||
- main | ||
|
||
jobs: | ||
call-workflow-passing-data: | ||
if: github.event.pull_request.merged | ||
uses: fivetran/dbt_package_automations/.github/workflows/auto-release.yml@main | ||
secrets: inherit | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,26 @@ | ||
database_key: zendesk_database | ||
schema_key: zendesk_schema | ||
|
||
dbt_versions: ">=1.3.0" "<2.0.0" | ||
|
||
table_variables: | ||
using_schedules: | ||
- daylight_time | ||
- schedule_holiday | ||
- schedule | ||
- ticket_schedule | ||
- time_zone | ||
using_domain_names: | ||
- domain_name | ||
using_user_tags: | ||
- user_tag | ||
using_ticket_form_history: | ||
- ticket_form_history | ||
using_organization_tags: | ||
- organization_tag | ||
|
||
destination_configurations: | ||
databricks: | ||
dispatch: | ||
- macro_namespace: dbt_utils | ||
search_order: [ 'spark_utils', 'dbt_utils' ] |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,19 @@ | ||
# dbt_zendesk v0.13.1 | ||
|
||
[PR #128](https://github.com/fivetran/dbt_zendesk/pull/128) includes the following changes: | ||
## Bug Fixes | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. should we note that this only affects people who use schedules in zendesk? might be implied bust just to be uber clear There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah that's a good clarification to include. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Updated. |
||
- The `int_zendesk__schedule_spine` model was updated to properly account for schedules that recognized daylight savings time (DST) at one point in time, and then stopped recognizing it at a later date. | ||
- For example, the Hong Kong timezone originally recognized DST, but then stopped in 1979. The previous versions of this package only recorded the schedule business hours until 1979. This update addresses this bug. | ||
- Please note, this update will only effect users leveraging schedules. | ||
|
||
## Under the Hood | ||
- Included auto-releaser GitHub Actions workflow to automate future releases. | ||
- Updated the maintainer PR template to resemble the most up to date format. | ||
- Included a `quickstart.yml` file to allow for automated Quickstart data model deployments. | ||
|
||
## Contributors | ||
- [@elijamuel](https://github.com/elijamuel) ([Issue #121](https://github.com/fivetran/dbt_zendesk/issues/121)) | ||
|
||
# dbt_zendesk v0.13.0 | ||
|
||
## 🚨 Breaking Change (Snowflake users) 🚨 | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,5 @@ | ||
name: 'zendesk' | ||
version: '0.13.0' | ||
version: '0.13.1' | ||
|
||
|
||
config-version: 2 | ||
|
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
Large diffs are not rendered by default.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just added the secret