-
Notifications
You must be signed in to change notification settings - Fork 73
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
Skip missing cards/dashboards in exposures #238
Conversation
…ps it instead of raising an error.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please read CONTRIBUTING.md for local checks to run before committing, to avoid waiting for failed CI.
CI has been tested locally (fmt-check and unit tests) |
Please read CONTRIBUTING.md carefully. You only ran the formatting check, you have to run all of them. |
* Skip missing cards/dashboards in exposures (#238) * In case of missing resource {dashboard, card, user}, dbt-metabase skips it instead of raising an error. * Rename get_[card, dashboard] to find_[card, dashboard], and remove duplicate return. * Fix unit test, change all occurences for get_dashboard --------- Co-authored-by: pierre dosne <pierre.dosne@fintecture.com> * Fix type and format issues * Add missing card to test * Formatting and imports --------- Co-authored-by: dosnep <dosnep@hotmail.fr> Co-authored-by: pierre dosne <pierre.dosne@fintecture.com>
Closes #237