move snackbar error message to utils#678
Conversation
| logger.error("You may need to update your workflow yaml file see FAQ") | ||
| logger.error("https://github.com/oist/optinist/wiki/FAQ") | ||
| logger.error(f"Invalid Workflow yaml param: [{key}]. See {faq_url}") | ||
| raise KeyError("Workflow yaml error, see FAQ") |
There was a problem hiding this comment.
Previously, if a key error occurred here, it would return a 422 error, but in the latest version, doesn't it return a 500 error?
Please recheck the behavior and record the test results.
There was a problem hiding this comment.
To confirm, I replaced if key not in default_params: on line 18 with if key in default_params: to force it to work (raise KeyError always occurs).
The result of the above seems to be a 500 error.
*If you still can't reproduce the issue, let's review it in detail using screen sharing, etc.
There was a problem hiding this comment.
@milesAraya
Have you tested any of the following?
- RUN ALL
- RUN
In the case of 2. RUN, is the snackbar for error 422 displayed properly?
There was a problem hiding this comment.
It is only possible to use RUN ALL when importing a yaml file.
2 participants
Fix for #629