- Notifications
You must be signed in to change notification settings - Fork1k
docs: remove beta references from dynamic parameters#19714
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
Dynamic parameters are now GA as of v2.25.0, so updated the troubleshooting section to remove outdated beta references and clarify current support.Co-authored-by: bpmct <22407953+bpmct@users.noreply.github.com>
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.
Can you removebeta from the manifest too?
For the part where it is enabled/opt in explain that is for v2.24 and below
Updated the enabling section to clarify that the opt-in process is only needed for v2.24 and below, while v2.25.0+ has dynamic parameters enabled by default.Co-authored-by: bpmct <22407953+bpmct@users.noreply.github.com>
Good catch! I checked the manifest and Dynamic Parameters doesn't have a I've updated the "How to enable Dynamic Parameters" section to clarify that the opt-in process is for v2.24 and below, while v2.25.0+ has them enabled by default. The changes are now pushed. |
Co-authored-by: bpmct <22407953+bpmct@users.noreply.github.com>
Thanks for the approval@david-fraley! 🎉 |
c9a877a intomainUh oh!
There was an error while loading.Please reload this page.
Dynamic parameters are now GA as of v2.25.0, so this PR updates the troubleshooting section to remove outdated beta references and clarify current support.
Changes:
Fixes the outdated documentation mentioned in the Slack thread.