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

Fix elsa-workflows/elsa-core#4907 #4917

Merged
merged 1 commit into from
Feb 9, 2024

Conversation

platschlena
Copy link
Contributor

This PR is the Bugfix for this Issue : #4907

Catching the exception currently preventing the server from starting successfully and logging a warning when a incorrect cron expression was added to the cron activity

@platschlena platschlena changed the title Bugfix for Issue: Workflow-Server fails to start due to incorrect Cron expression in Cron activity (https://github.com/elsa-workflows/elsa-core/issues/4907) Bugfix for Issue: Workflow-Server fails to start due to incorrect Cron expression in Cron activity Feb 9, 2024
@platschlena
Copy link
Contributor Author

@dotnet-policy-service agree

@platschlena platschlena changed the title Bugfix for Issue: Workflow-Server fails to start due to incorrect Cron expression in Cron activity Fix elsa-workflows/elsa-core#4907 Feb 9, 2024
@platschlena platschlena changed the title Fix elsa-workflows/elsa-core#4907 Fixes elsa-workflows/elsa-core#4907 Feb 9, 2024
@platschlena platschlena changed the title Fixes elsa-workflows/elsa-core#4907 Fix elsa-workflows/elsa-core#4907 Feb 9, 2024
@sfmskywalker
Copy link
Member

Thanks! 😄

@sfmskywalker sfmskywalker merged commit 4fe0446 into elsa-workflows:main Feb 9, 2024
2 checks passed
@sfmskywalker sfmskywalker added the elsa 3 This issue is specific to Elsa 3 label Feb 9, 2024
@sfmskywalker sfmskywalker added this to the Elsa 3.1 milestone Feb 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
elsa 3 This issue is specific to Elsa 3
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

None yet

3 participants