Skip to content

Pre release/add team variable to jira sprint enhanced#184

Open
fivetran-savage wants to merge 25 commits into
bugfix/remove_null_team_from_sprint_enhancedfrom
pre-release/add_team_variable_to_jira_sprint_enhanced
Open

Pre release/add team variable to jira sprint enhanced#184
fivetran-savage wants to merge 25 commits into
bugfix/remove_null_team_from_sprint_enhancedfrom
pre-release/add_team_variable_to_jira_sprint_enhanced

Conversation

@fivetran-savage
Copy link
Copy Markdown
Contributor

@fivetran-savage fivetran-savage commented Apr 14, 2026

Pre-release 1.9.0-a1

  • Creates variable to enable/disable team column at the model level for jira__sprint_enhanced
  • Fixes a previously existing bug that undercounted time fields in jira__sprint_enhanced

fivetran-savage and others added 17 commits April 14, 2026 14:16
* Integrity test sprint investigation (#177)

* Integrity test sprint investigation

* Revert dbt_project.yml

* schema

* Bugfix/jira using teams incremental fix (#180)

* bugfix/jira_using_teams_incremental_fix

* switch to append_new_columns

* Apply suggestions from code review

---------

Co-authored-by: Savage Fivetran <sarah.savage@fivetran.com>

* readme changelog

* changelog and on schema change fail

* changelog

* fix casting

* remove on schema change

* readme

* Uncomment Databricks SQL Warehouse test step

* Generate dbt docs via GitHub Actions

* Update CHANGELOG.md

---------

Co-authored-by: Avinash Kunnath <108772760+fivetran-avinash@users.noreply.github.com>
Co-authored-by: Savage Fivetran <sarah.savage@fivetran.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@fivetran-savage fivetran-savage marked this pull request as ready for review April 15, 2026 22:03
@fivetran-savage fivetran-savage added docs:ready Triggers the docs generator workflow. pre-release Triggers the auto-releaser workflow. labels Apr 16, 2026
@fivetran-savage fivetran-savage changed the base branch from main to bugfix/remove_null_team_from_sprint_enhanced April 16, 2026 17:15
Copy link
Copy Markdown
Contributor

@fivetran-jamie fivetran-jamie left a comment

Choose a reason for hiding this comment

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

Looks good, left a few suggestions

Comment thread .quickstart/quickstart.yml Outdated
Comment thread models/jira__sprint_enhanced.sql Outdated
Comment thread CHANGELOG.md Outdated
Comment thread dbt_project.yml Outdated
Comment thread README.md Outdated
jira_using_versions: false # Enabled by default. Disable if you do not have the versions table or do not want versions-related metrics reported.
jira_using_priorities: false # Enabled by default. Disable if you are not using priorities in Jira.
jira_using_teams: false # Enabled by default. Disable if you are not using teams in Jira.
jira_sprint_enhanced_include_teams: false # Enabled by default. Disable if you have the team table but want jira__sprint_enhanced at sprint-level granularity (one row per sprint instead of one row per team per sprint). Only applies when jira_using_teams is true.
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.

Since this section is more for source-table-affiliated variables, I think this variable may merit its own section (like jira_include_conversations)

fivetran-savage and others added 2 commits April 16, 2026 12:57
Co-authored-by: Jamie Rodriguez <65564846+fivetran-jamie@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@fivetran-jamie fivetran-jamie left a comment

Choose a reason for hiding this comment

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

Two requests but otherwise this is approved for pre-release! Will need to regen docs afterward

Comment thread models/jira__sprint_enhanced.sql Outdated
board_id,
sum(coalesce(original_estimate_seconds, 0)) as original_estimate_seconds,
sum(coalesce(remaining_estimate_seconds, 0)) as remaining_estimate_seconds,
sum(coalesce(time_spent_seconds), 0) as time_spent_seconds
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.

Suggested change
sum(coalesce(time_spent_seconds), 0) as time_spent_seconds
sum(coalesce(time_spent_seconds, 0)) as time_spent_seconds

Oops had a typo in my suggestion that caused BK to fail with function sum(integer, integer) does not exist

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.

That's on me. I pushed this late in the day and didn't wait for BK to finish before logging off.

Comment thread CHANGELOG.md
@@ -1,3 +1,22 @@
# dbt_jira v1.9.0-a1

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 add the PR

@fivetran-savage fivetran-savage added docs:ready Triggers the docs generator workflow. and removed docs:ready Triggers the docs generator workflow. labels Apr 17, 2026
@fivetran-savage fivetran-savage added pre-release Triggers the auto-releaser workflow. and removed pre-release Triggers the auto-releaser workflow. labels Apr 17, 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. pre-release Triggers the auto-releaser workflow.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants