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

feat: preserve alerts, update to 40sdk #97

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Prev Previous commit
Next Next commit
all owners for alert search
  • Loading branch information
michaelcramer committed Dec 2, 2022
commit ee4b846df05ffd3670dcd0014feea45a6ce77a18
2 changes: 1 addition & 1 deletion looker_deployer/commands/deploy_content.py
Original file line number Diff line number Diff line change
@@ -360,7 +360,7 @@ def main(args):

sdk = get_client(args.ini, args.env)
global enabled_alerts
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we can get an update to the readme about this new functionality

enabled_alerts = sdk.search_alerts(disabled="false")
enabled_alerts = sdk.search_alerts(disabled="false", all_owners=True)
send_content(
sdk,
args.env,