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(CanvasForm): Add support for ErrorHandler #955

Merged
merged 1 commit into from
Mar 21, 2024

Conversation

lordrip
Copy link
Member

@lordrip lordrip commented Mar 18, 2024

Context

Currently, there's partial support for oneOf in the uniforms library.

This commit adds a new OneOfField component that shows a selector to pick which schema should the form generate.

Also, if a model is already defined, the closest schema is selected automatically.

fix: #560
relates: #948
Reusing the internal ID will be implemented here: #962

@lordrip lordrip force-pushed the feat/support-error-handler branch from a43a9cd to 786a60c Compare March 19, 2024 08:32
@lordrip lordrip changed the title feat(CanvasForm): Add support for ErrorHandler [skip ci] feat(CanvasForm): Add support for ErrorHandler Mar 19, 2024
@@ -53,7 +54,9 @@ export class CamelRouteResource implements CamelResource, BeansAwareResource {

getVisualEntities(): CamelRouteVisualEntity[] {
return this.entities.filter(
(entity) => entity instanceof CamelRouteVisualEntity || entity instanceof CamelOnExceptionVisualEntity,
(entity) =>
entity instanceof CamelRouteVisualEntity ||
Copy link
Member Author

Choose a reason for hiding this comment

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

In the future, the idea is to remove this section by moving CamelRouteVisualEntity to the SUPPORTED_ENTITIES array, this way, we just need to update the SUPPORTED_ENTITIES array whenever there's a new Visual entity to support.

@lordrip lordrip force-pushed the feat/support-error-handler branch from 786a60c to 03aa5df Compare March 19, 2024 11:36
@lordrip lordrip force-pushed the feat/support-error-handler branch from 03aa5df to 813e266 Compare March 20, 2024 12:20
@lordrip lordrip marked this pull request as ready for review March 20, 2024 12:22
@lordrip lordrip requested review from tplevko, lhein and apupier March 20, 2024 12:24
Currently, there's partial support for `oneOf` in the `uniforms` library.

This commit adds a new `OneOfField` component that shows a selector to
pick which schema should the form generate.

Also, if a model is already defined, the closest schema is selected
automatically.

fix: KaotoIO#560
relates: KaotoIO#948
@lordrip lordrip force-pushed the feat/support-error-handler branch from 813e266 to 93f3eec Compare March 20, 2024 14:30
Copy link
Contributor

@tplevko tplevko left a comment

Choose a reason for hiding this comment

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

Found just two things I'm uncertain about.

@lordrip lordrip merged commit d6e068f into KaotoIO:main Mar 21, 2024
9 checks passed
@lordrip lordrip deleted the feat/support-error-handler branch March 21, 2024 09:03
@lordrip
Copy link
Member Author

lordrip commented Mar 21, 2024

Thanks for reviewing this PR @tplevko 💪

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.

Support Camel Route errorHandler
2 participants