Skip to content

Faces 4.1: Spec 1739 - Require firing events for @Initialized, @BeforeDestroyed, and @Destroyed for build-in scopes#5340

Merged
BalusC merged 1 commit intoeclipse-ee4j:masterfrom
volosied:spec1739
Oct 21, 2023
Merged

Faces 4.1: Spec 1739 - Require firing events for @Initialized, @BeforeDestroyed, and @Destroyed for build-in scopes#5340
BalusC merged 1 commit intoeclipse-ee4j:masterfrom
volosied:spec1739

Conversation

@volosied
Copy link
Copy Markdown
Contributor

@volosied volosied commented Oct 19, 2023

jakartaee/faces#1739

Both Mojarra and MyFaces should have this implemented. Now it's required via the faces spec.

The tests should also be added back in the next TCK release for 4.1 / 5.0. See https://github.com/jakartaee/faces/pull/1741/files

@volosied volosied changed the title Spec 1739: Require firing events for @Initialized, @BeforeDestroyed, and @Destroyed for build-in scopes Faces 4.1: Spec 1739 - Require firing events for @Initialized, @BeforeDestroyed, and @Destroyed for build-in scopes Oct 19, 2023
@BalusC BalusC added this to the 4.1.0 milestone Oct 21, 2023
@BalusC BalusC added the 4.1 label Oct 21, 2023
@BalusC BalusC merged commit dc7155f into eclipse-ee4j:master Oct 21, 2023
@BalusC
Copy link
Copy Markdown
Contributor

BalusC commented Oct 21, 2023

Oh! It was targeted at master which is 5.0. I'll have to backport. Can you target next 4.1 work at 4.1 branch? :)

@volosied
Copy link
Copy Markdown
Contributor Author

Sorry! I'll watch out for that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants