dbt-snowflake-monitoring
dbt-snowflake-monitoring copied to clipboard
Column Name mismatch in query_direct_table_access docs
In the query_direct_table_access model there is a mismatch between the docs and the model for table_name.
Happy to contribute a fix for this.
https://github.com/get-select/dbt-snowflake-monitoring/blob/6c440e1c2b7e3f32219e14c9cb7b0cabae5bcebe/models/query_direct_table_access.yml#L15-L16
https://github.com/get-select/dbt-snowflake-monitoring/blob/6c440e1c2b7e3f32219e14c9cb7b0cabae5bcebe/models/query_direct_table_access.sql#L6
If a fix is wanted, let me know what's preferred: table_name or full_table_name
Thanks @kirkbrunson-dave! A PR would be amazing. Let's update the docs to align with what is already in the model.
Thanks again for flagging.
Great! @ian-whitestone Do I need to do anything to get set up as a contributor? Getting the following pushing the PR to a feature branch
@kirkbrunson-dave it looks like an issue with your git command
Once you have pushed your commit to your forked repo, you should be able to open a PR from the github UI https://jarv.is/notes/how-to-pull-request-fork-github/