Uh oh!
There was an error while loading. Please reload this page.
Release 1149.0.0 - #9657
Conversation
The create-release-branch tool picked up the housekeeping fix commit and added it as an Uncategorized entry in the 17.2.0 section. Remove it since it is not a consumer-facing change.
❌ Changelog validation failed. Check the workflow run for details. |
There was a problem hiding this comment.
I am not sure if this file change belongs to the release PR but approving since this is just a comment change.
There was a problem hiding this comment.
They're only docstrings that were not updated.
There was a problem hiding this comment.
I won't block on this, but yes, ideally release PRs should only bump versions and dependencies and update changelogs, they should not update anything else.
There was a problem hiding this comment.
I won't block on this, but yes, ideally release PRs should only bump versions and dependencies and update changelogs, they should not update anything else.
Explanation
Releases
@metamask/ramps-controller@17.2.0(minor bump).Changes in this release:
getHeadlessProviderAllowlist(remoteFeatureFlagState)helper resolving the provider-id allowlist from themoneyHeadlessAllProvidersflag's object payload.HEADLESS_ALL_PROVIDERS_FEATURE_VERSIONconstant ('1') andgetHeadlessAllProvidersMinimumVersion(remoteFeatureFlagState)helper for version gating.moneyHeadlessAllProvidersflag value contract to accept{ enabled: true, featureVersion: "1", providerIds?: string[] }alongside the existing boolean form. Old clients coerce the object payload tofalse(native-only), so serving it cannot turn widening on for a client that cannot parse it.Also includes a housekeeping fix to the CHANGELOG and a JSDoc correction (stale
surface/HEADLESS_ALLOWLIST_SURFACES/HeadlessAllowlistSurfacereferences that were in the original #9524 merge before those were removed).References
moneyHeadlessAllProviderspayload carrying provider allowlists #9524 (moneyHeadlessAllProvidersobject payload support)Checklist
Note
Low Risk
Version bumps and documentation/changelog fixes only; no new logic in the diff.
Overview
Release cut for
@metamask/ramps-controller@17.2.0and the monorepo at1149.0.0, withtransaction-pay-controllerand the lockfile bumped to depend on ramps-controller^17.2.0.The 17.2.0 changelog is finalized and corrected so it matches the shipped API: object-form
moneyHeadlessAllProvidersis documented as{ enabled, featureVersion: "1", providerIds? }with helpersgetHeadlessProviderAllowlistandgetHeadlessAllProvidersMinimumVersion, and stale references to per-surface allowlists (surfaces,HEADLESS_ALLOWLIST_SURFACES,getQuotessurfaceoption) are removed.featureFlags.tsJSDoc is updated the same way. No runtime code changes appear in this diff beyond version numbers and docs.Reviewed by Cursor Bugbot for commit c6d5c11. Bugbot is set up for automated code reviews on this repo. Configure here.