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 <lukaszlen...@apache.org> wrote:
>
> sob., 13 lip 2024 o 10:23 Kusal Kithul-Godage
> <kusal.kithulgod...@gmail.com> 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: dev-unsubscr...@struts.apache.org
> For additional commands, e-mail: dev-h...@struts.apache.org
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@struts.apache.org
For additional commands, e-mail: dev-h...@struts.apache.org

Reply via email to