Skip to content

added cdi/interceptors-prioriry#185

Merged
arun-gupta merged 3 commits intojavaee-samples:masterfrom
rhanus:master
Jan 3, 2014
Merged

added cdi/interceptors-prioriry#185
arun-gupta merged 3 commits intojavaee-samples:masterfrom
rhanus:master

Conversation

@rhanus
Copy link
Contributor

@rhanus rhanus commented Jan 3, 2014

to demonstrate using Priority annotation on interceptor classes

@rhanus
Copy link
Contributor Author

rhanus commented Jan 3, 2014

Can you remove Oracle copyright from newly added files ?
Change the module name to name interceptors-priority ?
done sorry for typos

Do you want to move it to "interceptors" module instead of "cdi" ?
I think it's fine to have these tests here in cdi because of using @InterceptorBinding which is specific to CDI
I'm going to add a similar test into ejb in a moment where I will demonstrate use of @interceptors which is EJB specific

@rhanus
Copy link
Contributor Author

rhanus commented Jan 3, 2014

In the interceptor top-level module I'm going to add samples for life-cycle annotations and common interceptor aspects

arun-gupta added a commit that referenced this pull request Jan 3, 2014
added cdi/interceptors-prioriry
@arun-gupta arun-gupta merged commit 624d2f9 into javaee-samples:master Jan 3, 2014
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.

2 participants

Comments