You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by "mikoskinen (via GitHub)" <gi...@apache.org> on 2023/06/08 06:24:55 UTC

[GitHub] [camel-karavan] mikoskinen opened a new issue, #787: VS Code: Designer remove "intercept" section from YAML

mikoskinen opened a new issue, #787:
URL: https://github.com/apache/camel-karavan/issues/787

   The VS Code designer doesn't seem to yet have support for intercept. The main problem is that if intercept is manually added to the yaml-file, the designer completely removes.
   
   Steps to reproduce:
   1. Create route using designer
   2. Add intercept into the top of the yaml-file and save file
   3. Re-open the designer
   4. Make a change using the designer
   5. Notice how intercept is gone from the yaml-file


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] mgubaidullin commented on issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "mgubaidullin (via GitHub)" <gi...@apache.org>.
mgubaidullin commented on issue #787:
URL: https://github.com/apache/camel-karavan/issues/787#issuecomment-1594870785

   depends on: #798 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] mgubaidullin commented on issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "mgubaidullin (via GitHub)" <gi...@apache.org>.
mgubaidullin commented on issue #787:
URL: https://github.com/apache/camel-karavan/issues/787#issuecomment-1589250517

   @mikoskinen Karavan designer uses explicit YAML DSL. Yes, you need to have `routeConfiguration` and `route` on the top level.  


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] thewalk commented on issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "thewalk (via GitHub)" <gi...@apache.org>.
thewalk commented on issue #787:
URL: https://github.com/apache/camel-karavan/issues/787#issuecomment-1590427991

   > have u tried this yaml in a java app, which camel version? I tried this in my test env with camel version 3.20.0, it did not work, still got the error message as you mentioned above "Node should have only have 1 key, was: 2 keys: [id,steps] (Maybe this is an indent problem in the YAML source)."
   > 
   > > Manually editing the YAML to this seems to have worked:
   > > ```
   > > - routeConfiguration:
   > >     intercept:
   > >       - id: intercept-6d50
   > >         intercept:
   > >           id: intercept-233
   > >           steps:
   > >             - log:
   > >                 message: 'Intercepted: ${body}'
   > > ```
   > > 
   > > 
   > >     
   > >       
   > >     
   > > 
   > >       
   > >     
   > > 
   > >     
   > >   
   > > The designer doesn't render it but the good thing is that the designer doesn't seem to be deleting anything.
   
   below yaml should work for me. 'id' field made it a bad parsing.
   
   `
   - routeConfiguration:
       intercept:
         - intercept:
             steps:
               - log:
                   message: 'Intercepted: ${body}'
   `


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] mikoskinen commented on issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "mikoskinen (via GitHub)" <gi...@apache.org>.
mikoskinen commented on issue #787:
URL: https://github.com/apache/camel-karavan/issues/787#issuecomment-1581969971

   But! It seems the designer has a support for adding intercept using route configuration! That is great, and perfect workaround in my situation. 
   
   So maybe close this, what do you think? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] thewalk commented on issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "thewalk (via GitHub)" <gi...@apache.org>.
thewalk commented on issue #787:
URL: https://github.com/apache/camel-karavan/issues/787#issuecomment-1588835442

   have u tried this yaml in a java app, which camel version?
   I tried this in my test env with camel version 3.20.0, it did not work, still got the error message as you mentioned above "Node should have only have 1 key, was: 2 keys: [id,steps] (Maybe this is an indent problem in the YAML source)."


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] mikoskinen commented on issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "mikoskinen (via GitHub)" <gi...@apache.org>.
mikoskinen commented on issue #787:
URL: https://github.com/apache/camel-karavan/issues/787#issuecomment-1581986445

   Manually editing the YAML to this seems to have worked:
   
   ```
   - routeConfiguration:
       intercept:
         - id: intercept-6d50
           intercept:
             id: intercept-233
             steps:
               - log:
                   message: 'Intercepted: ${body}'
   ```
   The designer doesn't render it but the good thing is that the designer doesn't seem to be deleting anything.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [camel-karavan] mgubaidullin closed issue #787: VS Code: Designer removes "intercept" section from YAML

Posted by "mgubaidullin (via GitHub)" <gi...@apache.org>.
mgubaidullin closed issue #787: VS Code: Designer removes "intercept" section from YAML
URL: https://github.com/apache/camel-karavan/issues/787


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org