Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

implement NameConverterInterface instead of AdvancedNameConverterInte… #6956

Merged
merged 1 commit into from
Feb 10, 2025

Conversation

tacman
Copy link
Contributor

@tacman tacman commented Feb 7, 2025

…rface

Q A
Branch? current stable
License MIT
Issue Fixes #6948

Fix deprecation.

Per #6948 (comment)

@soyuka
Copy link
Member

soyuka commented Feb 7, 2025

could you run php-cs-fixer? It looks okay, maybe that I can even merge that to 4.1 as it looks fine using lowest dependencies!

@tacman
Copy link
Contributor Author

tacman commented Feb 7, 2025

done!

@soyuka soyuka changed the base branch from main to 4.1 February 8, 2025 12:24
@soyuka soyuka changed the base branch from 4.1 to main February 8, 2025 12:24
@soyuka soyuka force-pushed the remove-advanced-name branch from 5f61f78 to c463dd6 Compare February 10, 2025 08:55
@soyuka soyuka changed the base branch from main to 4.1 February 10, 2025 08:55
@soyuka soyuka merged commit d13203f into api-platform:4.1 Feb 10, 2025
79 checks passed
@soyuka
Copy link
Member

soyuka commented Feb 10, 2025

thanks @tacman this will be out soon

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.

AdvancedNameConverterInterface (in HydraPrefixNameConverter) is deprecated
2 participants