Skip to content
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

[Core] Jinja context variables versus dbt Jinja functions #5119

Open
dbeatty10 opened this issue Mar 19, 2024 · 2 comments
Open

[Core] Jinja context variables versus dbt Jinja functions #5119

dbeatty10 opened this issue Mar 19, 2024 · 2 comments
Labels
content Improvements or additions to content dbt Core The changes proposed in this issue relate to dbt Core size: medium This change will take up to a week to address

Comments

@dbeatty10
Copy link
Contributor

Link to the page(s) on docs.getdbt.com requiring updates

This page is a mix of documentation for functions and variables:
https://docs.getdbt.com/reference/dbt-jinja-functions

But the page title only mentions "dbt Jinja functions" and does not include "Jinja context variables".

Also, the base URL includes /dbt-jinja-functions/ which feels odd for a page like the schemas context variable (which is not a function):
https://docs.getdbt.com/reference/dbt-jinja-functions/schemas

Tell us more about this update

See above.

Reviewers/Stakeholders/SMEs

😎

Related GitHub issues

Here's two other docs issues that are tangentially related:

Additional information

No response

@dbeatty10 dbeatty10 added content Improvements or additions to content dbt Core The changes proposed in this issue relate to dbt Core labels Mar 19, 2024
@nataliefiann
Copy link
Contributor

Hiya @dbeatty10

Happy Tuesday, I hope you're well.

Can you kindly let me know what you'd like updating please?

Kind Regards
Natalie

@nataliefiann nataliefiann added the size: medium This change will take up to a week to address label Sep 17, 2024
@dbeatty10
Copy link
Contributor Author

The big idea is delineating the Jinja functions from the Jinja context variables. Right now, the context variables are all just lumped in with the functions (but they aren't functions!).

We've got a couple main options:

  1. Leave them all together, but update the page title and URL to include both "functions" and "variables"
  2. Create a new page and move the context variables to that page

If we choose option 1, the most simple approach would to be rename the title of the page from "dbt Jinja functions" to "dbt Jinja functions and context variables".

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content dbt Core The changes proposed in this issue relate to dbt Core size: medium This change will take up to a week to address
Projects
None yet
Development

No branches or pull requests

2 participants