eclipse-ee4j / mojarra

Mojarra, a Jakarta Faces implementation
Other
166 stars 112 forks source link

Support new FacesMessage.Severity.SUCCESS in h:message/h:messages #5490

Closed BalusC closed 2 months ago

BalusC commented 2 months ago

https://github.com/eclipse-ee4j/mojarra/issues/5489

Associated PR in Faces side: https://github.com/jakartaee/faces/pull/1964

mnriem commented 2 months ago

I would rather see Severity.NONE instead of SUCCESS.

BalusC commented 2 months ago

That's not what was requested by community https://github.com/jakartaee/faces/issues/1592.

mnriem commented 2 months ago

I realize that, but it seems an odd fit.

Or asking a nice LLM to explain message severity and "SUCCES" to me:

'In the context of message severity, “SUCCESS” is generally not considered a severity level. Severity levels typically indicate the urgency or criticality of an issue, ranging from emergency situations to debug information. “SUCCESS” is more of a status or outcome, indicating that an operation or process completed as intended without errors'

BalusC commented 2 months ago

Please continue in https://github.com/jakartaee/faces/issues/1592