Skip to content

fix(di): use correct parameter name for validation group#206

Merged
Jibbarth merged 1 commit intosynolia:1.xfrom
delyriand:fix/205-use-correct-parameter
Aug 12, 2025
Merged

fix(di): use correct parameter name for validation group#206
Jibbarth merged 1 commit intosynolia:1.xfrom
delyriand:fix/205-use-correct-parameter

Conversation

@delyriand
Copy link

Q A
Bug fix? yes
New feature? no
BC breaks? no
Fixed issue #205

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@maxperei
Copy link

hi @delyriand, yes this was fixed only on branch main and should be backported on 1.x but someone has to do it now ^^
so it has already be done in : 8e8ae37

@delyriand
Copy link
Author

Hello @maxperei! Thank you for your quick response.
I will try to use your commit as a patch from my composer while waiting for a new tag 4.1.

And I am closing this PR!

@delyriand delyriand closed this Jul 31, 2025
@Jibbarth Jibbarth reopened this Aug 12, 2025
@Jibbarth
Copy link
Member

Hello @delyriand

Let's merge this one on 1.x compat branch. As it was previously that parameters on 4.0, this patch is correct.

Thank you 🙏

@Jibbarth Jibbarth merged commit 32541b6 into synolia:1.x Aug 12, 2025
2 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants