Ah just saw this - I've already created a fix PR, took me a little
longer than expected as it turns out there were a few more broken
Showcase Actions which I've also fixed
https://github.com/apache/struts/pull/986

On Sat, Jul 13, 2024 at 7:23 PM Lukasz Lenart <[email protected]> wrote:
>
> sob., 13 lip 2024 o 10:23 Kusal Kithul-Godage
> <[email protected]> napisał(a):
> > That's correct, it's only enabled by default from 7.0, but I enabled
> > it manually for the Showcase App so we can ensure its functionality
> > and catch regressions. It seems in this case, we have an Action that
> > utilises the Convention plugin but isn't actually covered by any
> > tests.
>
> I can add a test to cover this action plus I will add @StrutsParameter
> where needed
>
> Cheers
> Lukasz
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to