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

[CT-1990] Parse access property on model node #6824

Closed
Tracked by #6747
MichelleArk opened this issue Feb 1, 2023 · 0 comments · Fixed by #7007
Closed
Tracked by #6747

[CT-1990] Parse access property on model node #6824

MichelleArk opened this issue Feb 1, 2023 · 0 comments · Fixed by #7007
Assignees
Labels
model_groups_access Issues related to groups multi_project user docs [docs.getdbt.com] Needs better documentation

Comments

@MichelleArk
Copy link
Contributor

MichelleArk commented Feb 1, 2023

Requirements

  • access is a new property on model nodes. For phase 1, cannot be provided for any other 'groupable' nodes
  • access property should be defined as an enum, for potential extension in the future. Currently must be one of private or public, or protected:
  • private - ref works within the same group
  • protected - ref works within the same project/package
  • public - ref works across projects/packages
  • access defaults to protected, irrespective of the group config value

Reference spec

# models/marts/github/github.yml

groups:
  - name: github
    owner: 
      name: Jeremy
      email: data@jer.co

models:
  - name: int__github_issue_label_history
    access: private
    config: 
      group: github  # explicit opt-in
  - name: fct_github_issues
    access: protected
    config: 
      group: github  # explicit opt-in
  - name: fct_github_issues
    access: public
@github-actions github-actions bot changed the title Parse optional access property on model node [CT-1990] Parse optional access property on model node Feb 1, 2023
@gshank gshank self-assigned this Feb 14, 2023
@MichelleArk MichelleArk changed the title [CT-1990] Parse optional access property on model node [CT-1990] Parse access property on model node Feb 14, 2023
@MichelleArk MichelleArk added the user docs [docs.getdbt.com] Needs better documentation label Feb 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
model_groups_access Issues related to groups multi_project user docs [docs.getdbt.com] Needs better documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants