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

Allow force save with syntax error #932

Merged
merged 3 commits into from
Oct 23, 2023
Merged

Conversation

jw3
Copy link
Member

@jw3 jw3 commented Oct 22, 2023

Allows config to be saved even if there is a syntax error.

The UI displays a message indicating there is a syntax error and to proceed only if sure, as is already done in the rule editor. This also updates the text used in the message to be generic enough to use across rules and config.

Closes #931

@jw3 jw3 merged commit 28f7930 into ctc-oss:master Oct 23, 2023
@jw3 jw3 deleted the ui/config_navigate_away branch October 23, 2023 10:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow config save override in case of error
1 participant