Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@pimmey
Copy link
Contributor

@pimmey pimmey commented Dec 18, 2025

Closes: PROD-1985 / LIGHTDASH-BACKEND-BDE

Description:

Skip Slack channel sync job when Slack is not configured. This guards the job from execution and logs a message indicating why the sync was skipped. This way we prevent further execution down the codepath that would throw MissingConfigError: Slack is not configured.

Copy link
Contributor Author

pimmey commented Dec 18, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@pimmey pimmey self-assigned this Dec 18, 2025
@pimmey pimmey requested a review from joaoviana December 18, 2025 15:58
@pimmey pimmey marked this pull request as ready for review December 18, 2025 15:58
@linear
Copy link

linear bot commented Dec 18, 2025

@github-actions
Copy link

Your preview environment pr-18959 has been deployed.

Preview environment endpoints are available at:

@github-actions
Copy link

Preview Environment

🌐 URL: https://lightdash-preview-pr-18959.lightdash.okteto.dev

πŸ“‹ Logs: View in GCP Console

πŸ”§ SSH: ./scripts/okteto-ssh.sh 18959

@pimmey pimmey merged commit fdab50d into main Dec 18, 2025
16 checks passed
@pimmey pimmey deleted the MissingConfigError-Slack-is-not-configured branch December 18, 2025 16:12
lightdash-bot pushed a commit that referenced this pull request Dec 18, 2025
## [0.2265.3](0.2265.2...0.2265.3) (2025-12-18)

### Bug Fixes

* guard GENERATE_SLACK_CHANNEL_SYNC_JOBS if slack client is not configured ([#18959](#18959)) ([fdab50d](fdab50d))
@lightdash-bot
Copy link
Collaborator

πŸŽ‰ This PR is included in version 0.2265.3 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants