Update sl-jdbc.md #4604

rpourzand · 2023-12-07T00:52:09Z

We're making some changes to our API (not released yet), so I'm preemptively adding info.

What are you changing in this pull request and why?

Checklist

Review the Content style guide so my content adheres to these guidelines.
For docs versioning, review how to version a whole page and version a block of content.
Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

Adding new pages (delete if not applicable):

Add page to website/sidebars.js
Provide a unique filename for the new page

Removing or renaming existing pages (delete if not applicable):

Remove page from website/sidebars.js
Add an entry website/static/_redirects
Run link testing locally with npm run build to update the links that point to the deleted page

We're making some changes to our API (not released yet), so I'm preemptively adding info.

vercel · 2023-12-07T00:52:15Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs-getdbt-com	🛑 Canceled (Inspect)			Jan 23, 2024 11:40am

DevonFulcher

Looks good, based on Paul's demo. I'm not sure when these features go live though.

Adding an example from Paul

copying the same over for jdbc -- maybe we can figure out a way to put this somewhere so we don't have to duplicate

timedimension is no longer optional

fixing some updates

fixing typo

plypaul

LGTM aside form comments

website/docs/docs/dbt-cloud-apis/sl-graphql.md

website/docs/docs/dbt-cloud-apis/sl-jdbc.md

plypaul · 2023-12-08T19:45:50Z

The ambiguous resolution stack has not been released yet, so do we need to wait for that to go out before merging this?

making paul's reco's

paul's recos

also adding a dimension only example to it

website/docs/docs/dbt-cloud-apis/sl-jdbc.md

Jstein77

This looks right to me. I think we need to add some more examples of how to query multi-hop dimensions in the CLI - but I can add that in a different PR.

rpourzand · 2024-01-23T06:53:20Z

@mirnawong1 this is ready to merge!

website/docs/docs/dbt-cloud-apis/sl-jdbc.md

website/docs/docs/dbt-cloud-apis/sl-graphql.md

mirnawong1 · 2024-01-23T11:34:20Z

double checked and looking great! merging 🚢 - thank you so much @rpourzand !

Update sl-jdbc.md

e619ffa

We're making some changes to our API (not released yet), so I'm preemptively adding info.

rpourzand requested a review from a team as a code owner December 7, 2023 00:52

rpourzand requested review from plypaul and DevonFulcher December 7, 2023 00:52

github-actions bot added content Improvements or additions to content size: x-small This change will take under 3 hours to fix. labels Dec 7, 2023

vercel bot deployed to Preview December 7, 2023 00:58 View deployment

Merge branch 'current' into rpourzand-patch-api-updates

7dafb5f

vercel bot deployed to Preview December 7, 2023 12:00 View deployment

DevonFulcher approved these changes Dec 7, 2023

View reviewed changes

Merge branch 'current' into rpourzand-patch-api-updates

4844dd3

vercel bot deployed to Preview December 7, 2023 18:29 View deployment

Update sl-jdbc.md

dc49f19

Adding an example from Paul

github-actions bot added size: medium This change will take up to a week to address and removed size: x-small This change will take under 3 hours to fix. labels Dec 8, 2023

rpourzand added 4 commits December 7, 2023 17:57

Update sl-graphql.md

be5c52c

copying the same over for jdbc -- maybe we can figure out a way to put this somewhere so we don't have to duplicate

Update sl-jdbc.md

8c510b7

timedimension is no longer optional

Update sl-jdbc.md

436577f

fixing some updates

Update sl-graphql.md

c250bb6

fixing typo

vercel bot deployed to Preview December 8, 2023 02:09 View deployment

plypaul approved these changes Dec 8, 2023

View reviewed changes

website/docs/docs/dbt-cloud-apis/sl-graphql.md Outdated Show resolved Hide resolved

website/docs/docs/dbt-cloud-apis/sl-jdbc.md Outdated Show resolved Hide resolved

rpourzand added 2 commits December 8, 2023 14:52

Update sl-graphql.md

61af03e

making paul's reco's

Update sl-jdbc.md

c5afac9

paul's recos

vercel bot deployed to Preview December 8, 2023 23:06 View deployment

Merge branch 'current' into rpourzand-patch-api-updates

0b96df9

vercel bot deployed to Preview December 19, 2023 13:10 View deployment

Update sl-graphql.md

384eba8

also adding a dimension only example to it

vercel bot deployed to Preview December 19, 2023 19:06 View deployment

matthewshaver reviewed Dec 22, 2023

View reviewed changes

website/docs/docs/dbt-cloud-apis/sl-jdbc.md Outdated Show resolved Hide resolved

Apply suggestions from code review

7b8cc9a

vercel bot deployed to Preview December 22, 2023 19:29 View deployment

mirnawong1 added the do not merge label Dec 22, 2023

Merge branch 'current' into rpourzand-patch-api-updates

871dabf

vercel bot deployed to Preview January 9, 2024 18:18 View deployment

Jstein77 approved these changes Jan 19, 2024

View reviewed changes

rpourzand added 2 commits January 19, 2024 17:17

Update sl-graphql.md

6a32ba6

Update sl-graphql.md

1d05363

vercel bot deployed to Preview January 20, 2024 01:28 View deployment

Merge branch 'current' into rpourzand-patch-api-updates

d68dbf7

mirnawong1 reviewed Jan 23, 2024

View reviewed changes

website/docs/docs/dbt-cloud-apis/sl-jdbc.md Outdated Show resolved Hide resolved

Update website/docs/docs/dbt-cloud-apis/sl-jdbc.md

357e5aa

mirnawong1 reviewed Jan 23, 2024

View reviewed changes

website/docs/docs/dbt-cloud-apis/sl-jdbc.md Outdated Show resolved Hide resolved

Update website/docs/docs/dbt-cloud-apis/sl-jdbc.md

192c2f0

mirnawong1 reviewed Jan 23, 2024

View reviewed changes

website/docs/docs/dbt-cloud-apis/sl-jdbc.md Outdated Show resolved Hide resolved

Update website/docs/docs/dbt-cloud-apis/sl-jdbc.md

250b3ba

mirnawong1 reviewed Jan 23, 2024

View reviewed changes

website/docs/docs/dbt-cloud-apis/sl-graphql.md Outdated Show resolved Hide resolved

mirnawong1 added 3 commits January 23, 2024 11:30

Update website/docs/docs/dbt-cloud-apis/sl-graphql.md

fb3b8f6

Update sl-graphql.md

610da38

Update sl-jdbc.md

7f3cc44

mirnawong1 approved these changes Jan 23, 2024

View reviewed changes

mirnawong1 enabled auto-merge January 23, 2024 11:33

mirnawong1 removed the do not merge label Jan 23, 2024

vercel bot deployed to Preview January 23, 2024 11:40 View deployment

mirnawong1 merged commit d5e43da into current Jan 23, 2024
7 checks passed

mirnawong1 deleted the rpourzand-patch-api-updates branch January 23, 2024 11:40

runleonarun added the January-2024 label Feb 29, 2024

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Update sl-jdbc.md #4604

Update sl-jdbc.md #4604

rpourzand commented Dec 7, 2023

vercel bot commented Dec 7, 2023 •

edited

Loading

DevonFulcher left a comment

plypaul left a comment

plypaul commented Dec 8, 2023

Jstein77 left a comment •

edited

Loading

rpourzand commented Jan 23, 2024

mirnawong1 commented Jan 23, 2024

Update sl-jdbc.md #4604

Update sl-jdbc.md #4604

Conversation

rpourzand commented Dec 7, 2023

What are you changing in this pull request and why?

Checklist

vercel bot commented Dec 7, 2023 • edited Loading

DevonFulcher left a comment

Choose a reason for hiding this comment

plypaul left a comment

Choose a reason for hiding this comment

plypaul commented Dec 8, 2023

Jstein77 left a comment • edited Loading

Choose a reason for hiding this comment

rpourzand commented Jan 23, 2024

mirnawong1 commented Jan 23, 2024

vercel bot commented Dec 7, 2023 •

edited

Loading

Jstein77 left a comment •

edited

Loading