-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Restrict param type of AdminInterface identifiers methods #6311
Restrict param type of AdminInterface identifiers methods #6311
Conversation
35151c6
to
0c15593
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The deprecation path seems right to me. We need to fix the QA checks.
I have two solutions:
Theorically |
Could you please rebase your PR and fix merge conflicts? |
0c15593
to
e8d574e
Compare
e8d574e
to
b531c59
Compare
b531c59
to
019d7b0
Compare
Subject
I am targeting this branch, because BC.
I'm introducing deprecation related to this PR: #6304
Changelog