Set masking_policy_list should be outside of the loop about dbt-snow-mask HOT 6 CLOSED

@entechlog Thanks for the fix!
It's already what I was using on my side, using the package locally, so I can say it's working as expected indeed 🙂

from dbt-snow-mask.

Thank You @IL-William . I will do the release in a day or so from prerelease

from dbt-snow-mask.

@IL-William, I have just released a new version 0.2.0 and should be reflected in dbt hub in about an hour OR so. Closing the issue now.

from dbt-snow-mask.

Hello @IL-William , I will check this and get back.

from dbt-snow-mask.

Hey @entechlog
Sorry I didn't have a chance to look at this before!
Coming back on this, I think it could be a simple solution like that.

{# If there are some masking policies to be applied in this model, we should show the masking policies in the schema. Otherwise we don't need it. #}
        {% if meta_columns | length > 0 %}
            {% set masking_policy_list = dbt_utils.get_query_results_as_dict(masking_policy_list_sql) %}
        {% endif %}

from dbt-snow-mask.

@IL-William , Thanks for the suggestion. I have created release candidate https://github.com/entechlog/dbt-snow-mask/releases/tag/0.2.0rc1. Can you please test this and see if things are still looking good after the loop fix ?

from dbt-snow-mask.