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.

Sign up for GitHub

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

Jump to bottom

Lite/Full split: decide on "The EventMetadata interface" #493

Closed
Ladicek opened this issue May 20, 2021 · 1 comment · Fixed by #525
Closed

Lite/Full split: decide on "The EventMetadata interface" #493

Ladicek opened this issue May 20, 2021 · 1 comment · Fixed by #525
Assignees
@manovotn
Labels
Lite Related to CDI Lite

Comments

@Ladicek
Copy link
Contributor

Ladicek commented May 20, 2021

The "The EventMetadata interface" chapter has a TODO for deciding whether we should mention BeanManager.getEvent() or BeanContainer.getEvent().

https://github.com/eclipse-ee4j/cdi/blob/864e1b4f779e4d6d5a80f147d975a510446d4987/spec/src/main/asciidoc/core/events.asciidoc
@Ladicek Ladicek mentioned this issue May 20, 2021
Closed
12 tasks
@Ladicek Ladicek added the Lite Related to CDI Lite label May 24, 2021
@manovotn
Copy link
Contributor

Since getEvent() is now on BeanContainer, we should mention that one instead.

@manovotn manovotn self-assigned this Sep 13, 2021
@manovotn manovotn mentioned this issue Sep 13, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@manovotn manovotn

Labels
Lite Related to CDI Lite
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace BeanManager#getEvents with reference to BeanContainer. manovotn/cdi
2 participants
@Ladicek @manovotn