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

Simple CDI/Camel eventing bridge endpoints, see Javadoc of CdiEvent #6

Merged
merged 1 commit into from
Feb 22, 2015
Merged

Simple CDI/Camel eventing bridge endpoints, see Javadoc of CdiEvent #6

merged 1 commit into from
Feb 22, 2015

Conversation

christianbauer
Copy link
Contributor

This is already an improvement compared with the patch in https://issues.apache.org/jira/browse/CAMEL-5408

Next step would be to register ObserverMethods specific for all discovered @uri @SomeQualifier injection points.

Beyond that some parser/generator DSL would be nice, so you could specify "cdi-event://some.BeanType?qualifier=some.OtherType&qualifier=more.CustomQualifiers" as endpoint URL.

@coveralls
Copy link

Coverage Status

Coverage increased (+4.72%) to 82.43% when pulling 5c40908 on christianbauer:master into 34f1045 on astefanutti:master.

2 similar comments
@coveralls
Copy link

Coverage Status

Coverage increased (+4.72%) to 82.43% when pulling 5c40908 on christianbauer:master into 34f1045 on astefanutti:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+4.72%) to 82.43% when pulling 5c40908 on christianbauer:master into 34f1045 on astefanutti:master.

astefanutti added a commit that referenced this pull request Feb 22, 2015
Simple CDI/Camel eventing bridge endpoints, see Javadoc of CdiEvent
@astefanutti astefanutti merged commit a979a9a into astefanutti:master Feb 22, 2015
@astefanutti
Copy link
Owner

Thanks! I should be able to iterate on the next steps and have a new version release in the upcoming weeks.

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.

3 participants