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

Error with multiple words for owner name #74

Closed
1 of 5 tasks
graciegoheen opened this issue Jun 26, 2023 · 3 comments
Closed
1 of 5 tasks

Error with multiple words for owner name #74

graciegoheen opened this issue Jun 26, 2023 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@graciegoheen
Copy link
Collaborator

Describe the bug

Following the example from the docs site:

# create a group of all models tagged with "finance"
# leaf nodes and nodes with cross-group dependencies will be `public`
# public nodes will also have contracts added to them
dbt-meshify group finance --owner name Monopoly Man -s +tag:finance

I attempted to create a new group with a 2-word owner name and got an error:
Screen Shot 2023-06-26 at 11 04 29 AM

This might just be a docs issue (and we need double quotes around the owner name), but overall I think this points to us maybe re-thinking the UX!

System information

Which database are you using dbt with?

  • postgres
  • redshift
  • bigquery
  • snowflake
  • other (specify: ____________)

The output of dbt --version:

Core:
  - installed: 1.5.2
  - latest:    1.5.2 - Up to date!

Plugins:
  - snowflake: 1.5.2 - Up to date!

Additional context

Are you interested in contributing the fix?

@graciegoheen graciegoheen added bug Something isn't working triage Tis issue or pull request must be triaged by a project maintainer labels Jun 26, 2023
@nicholasyager
Copy link
Collaborator

@graciegoheen Depending on your version locally, you might be using the old owner syntax. Please see this PR for more details.

@graciegoheen
Copy link
Collaborator Author

Thanks, looks like I'm on the old owner syntax. Thoughts on moving to a "wizard" experience - with CLI prompts to add information (or supporting both). I feel like remembering all of the appropriate arg syntax is a bit clunky.

@nicholasyager nicholasyager removed the triage Tis issue or pull request must be triaged by a project maintainer label Jul 10, 2023
@nicholasyager nicholasyager self-assigned this Jul 10, 2023
@nicholasyager
Copy link
Collaborator

Resolved in #72

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants