-
-
Notifications
You must be signed in to change notification settings - Fork 9.8k
[Serializer] Deprecate AdvancedNameConverterInterface
#58308
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
[Serializer] Deprecate AdvancedNameConverterInterface
#58308
Conversation
|
That interface looks indeed useless since 7.1. @xabbuh can you have a look as you were the one making NameConverterInterface methods' signatures the same as the |
|
👍 looks good to me |
e5ec432 to
ac0f41f
Compare
|
@chalasr, the |
|
Can you please rebase now that #58368 is merged to ensure tests keep passing despite an implementation exists in core? |
ac0f41f to
0a263d0
Compare
|
Thank you @mtarld. |
Deprecate the
AdvancedNameConverterInterfacein favor ofNameConverterInterfaceas they'll have the same signature in 8.0