-
Notifications
You must be signed in to change notification settings - Fork 578
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
[Plugin Hub] Display deprecated fields #7376
Conversation
✅ Deploy Preview for kongdocs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
For shorthand fields, this makes sense. But looks like they're now listing the field mapping info directly in that deprecation notice. Maybe we should rephrase this? Something like: "If the deprecation notice for a parameter doesn't include new field mapping info, see the changelog." |
d936762
to
4b463b6
Compare
As of version 3.7.x, fields in the plugin schema have been flagged with a `deprecation` key, containing information about the deprecation. This commit adds the fields marked as deprecated to the list of `Deprecated Parameters` on the Configuration Reference page. It also renders a deprecation notice, with the information provided in the field. Note: Fields marked as deprecated are not rendered in the main body of the schema, just in the `Deprecated Parameters` section.
And add a `Deprecation notice` to shorthand fields if they don't have one.
4b463b6
to
ee56773
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.
Couple of minor phrasing suggestions, otherwise lgtm
Co-authored-by: lena-larionova <54370747+lena-larionova@users.noreply.github.com>
Co-authored-by: lena-larionova <54370747+lena-larionova@users.noreply.github.com>
Description
As of version 3.7.x, fields in the plugin schema have been flagged with a
deprecation
key, containing information about the deprecation.deprecation notice
on deprecated fieldsDeprecated Parameters
noticedeprecation notice
toshorthand_fields
if they don't have one.Testing instructions
datadog - fields marked as deprecated
session - shorthand fields only (these weren't marked as deprecated)
forward-proxy - shorthand fields were marked as deprecated
Checklist