Skip to content

feat: add subscription discountType - #10560

Merged
lwin-kyaw merged 3 commits into
mainfrom
feat/swap-subscription-discount
Sep 29, 2026
Merged

lwin-kyaw merged 3 commits into
mainfrom
feat/swap-subscription-discount

Conversation

@lwin-kyaw

@lwin-kyaw lwin-kyaw commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Explanation

Add DiscountType.SUBSCRIPTION ('subscription') to the bridge quote fee model so subscription-based MetaBridge discounts can be represented alongside the existing vip, promo, and dao values.

The quote validator already accepts optional string discount types, so this is an additive typed constant with no changes to quote processing or fee calculations.

References

None.

Test plan

  • Added validator coverage for DiscountType.SUBSCRIPTION
  • Bridge-controller test suite passes
  • Changelog validation passes
  • No linter errors

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation and the package changelog
  • I've communicated my changes to consumers by updating the changelog
  • I've introduced breaking changes

Note

Low Risk
Additive enum export and tests only; quote validation and fee logic are unchanged.

Overview
Adds DiscountType.SUBSCRIPTION ('subscription') to the exported quote fee enum so clients can recognize subscription-based MetaBridge discounts alongside vip, promo, and dao.

FeeDataSchema still treats discountType as an optional string, so this is a typed constant and changelog entry only—no validator or fee-calculation behavior changes. Tests cover the new enum value in FeeDataSchema acceptance cases and assert the string value 'subscription'.

Reviewed by Cursor Bugbot for commit 03b7eb1. Bugbot is set up for automated code reviews on this repo. Configure here.

@lwin-kyaw
lwin-kyaw requested review from a team as code owners September 29, 2026 04:54
@lwin-kyaw
lwin-kyaw deployed to default-branch September 29, 2026 04:54 — with GitHub Actions Active
@lwin-kyaw

Copy link
Copy Markdown
Contributor Author

@metamaskbot publish-preview

@github-actions

Copy link
Copy Markdown
Contributor

Preview builds have been published. Learn how to use preview builds in other projects.

Expand for full list of packages and versions.
@metamask-previews/account-tree-controller@11.0.0-preview-0898196
@metamask-previews/accounts-controller@40.0.0-preview-0898196
@metamask-previews/address-book-controller@8.0.0-preview-0898196
@metamask-previews/advanced-chart-core@0.0.0-preview-0898196
@metamask-previews/ai-controllers@2.0.0-preview-0898196
@metamask-previews/analytics-controller@3.2.0-preview-0898196
@metamask-previews/analytics-data-regulation-controller@0.0.0-preview-0898196
@metamask-previews/announcement-controller@9.0.0-preview-0898196
@metamask-previews/app-metadata-controller@3.0.0-preview-0898196
@metamask-previews/approval-controller@10.0.0-preview-0898196
@metamask-previews/assets-controller@17.0.0-preview-0898196
@metamask-previews/assets-controllers@112.0.4-preview-0898196
@metamask-previews/authenticated-user-storage@4.1.0-preview-0898196
@metamask-previews/base-controller@10.0.0-preview-0898196
@metamask-previews/base-data-service@2.1.0-preview-0898196
@metamask-previews/bitcoin-regtest-up@2.0.0-preview-0898196
@metamask-previews/bridge-controller@81.3.3-preview-0898196
@metamask-previews/bridge-status-controller@76.3.3-preview-0898196
@metamask-previews/build-utils@4.0.0-preview-0898196
@metamask-previews/chain-agnostic-permission@2.0.0-preview-0898196
@metamask-previews/chomp-api-service@6.0.0-preview-0898196
@metamask-previews/claims-controller@1.0.2-preview-0898196
@metamask-previews/client-controller@2.0.0-preview-0898196
@metamask-previews/client-utils@3.0.3-preview-0898196
@metamask-previews/compliance-controller@3.0.0-preview-0898196
@metamask-previews/composable-controller@13.0.0-preview-0898196
@metamask-previews/config-registry-controller@4.0.0-preview-0898196
@metamask-previews/connectivity-controller@1.0.0-preview-0898196
@metamask-previews/controller-utils@13.0.0-preview-0898196
@metamask-previews/core-backend@11.0.0-preview-0898196
@metamask-previews/cryptography@0.0.0-preview-0898196
@metamask-previews/delegation-controller@4.0.0-preview-0898196
@metamask-previews/earn-controller@13.0.2-preview-0898196
@metamask-previews/eip-5792-middleware@4.0.1-preview-0898196
@metamask-previews/eip-7702-internal-rpc-middleware@1.0.0-preview-0898196
@metamask-previews/eip1193-permission-middleware@3.0.0-preview-0898196
@metamask-previews/eth-block-tracker@16.0.0-preview-0898196
@metamask-previews/eth-json-rpc-middleware@25.0.0-preview-0898196
@metamask-previews/eth-json-rpc-provider@7.0.0-preview-0898196
@metamask-previews/foundryup@2.0.0-preview-0898196
@metamask-previews/gas-fee-controller@27.0.0-preview-0898196
@metamask-previews/gator-permissions-controller@6.0.1-preview-0898196
@metamask-previews/geolocation-controller@2.0.0-preview-0898196
@metamask-previews/java-tron-up@2.0.0-preview-0898196
@metamask-previews/json-rpc-engine@11.0.0-preview-0898196
@metamask-previews/json-rpc-middleware-stream@9.0.0-preview-0898196
@metamask-previews/keyring-controller@28.1.0-preview-0898196
@metamask-previews/kyc-controller@0.6.0-preview-0898196
@metamask-previews/local-node-utils@2.0.0-preview-0898196
@metamask-previews/logging-controller@10.0.0-preview-0898196
@metamask-previews/message-manager@15.0.0-preview-0898196
@metamask-previews/messenger@3.0.0-preview-0898196
@metamask-previews/messenger-cli@1.0.0-preview-0898196
@metamask-previews/money-account-api-data-service@2.1.0-preview-0898196
@metamask-previews/money-account-balance-service@3.1.1-preview-0898196
@metamask-previews/money-account-controller@2.0.0-preview-0898196
@metamask-previews/money-account-upgrade-controller@5.1.0-preview-0898196
@metamask-previews/money-account-utils@2.1.0-preview-0898196
@metamask-previews/multichain-account-service@14.1.0-preview-0898196
@metamask-previews/multichain-api-middleware@5.0.0-preview-0898196
@metamask-previews/multichain-network-controller@4.0.0-preview-0898196
@metamask-previews/multichain-transactions-controller@8.0.0-preview-0898196
@metamask-previews/name-controller@10.0.0-preview-0898196
@metamask-previews/network-connection-banner-controller@1.0.0-preview-0898196
@metamask-previews/network-controller@37.0.0-preview-0898196
@metamask-previews/network-enablement-controller@7.0.1-preview-0898196
@metamask-previews/notification-services-controller@29.0.2-preview-0898196
@metamask-previews/passkey-controller@4.1.0-preview-0898196
@metamask-previews/permission-controller@14.0.0-preview-0898196
@metamask-previews/permission-log-controller@6.0.0-preview-0898196
@metamask-previews/perps-controller@18.0.1-preview-0898196
@metamask-previews/phishing-controller@18.1.1-preview-0898196
@metamask-previews/platform-api-docs@0.2.1-preview-0898196
@metamask-previews/polling-controller@17.0.0-preview-0898196
@metamask-previews/preferences-controller@24.0.0-preview-0898196
@metamask-previews/profile-metrics-controller@5.1.2-preview-0898196
@metamask-previews/profile-sync-controller@33.0.0-preview-0898196
@metamask-previews/ramps-controller@26.0.1-preview-0898196
@metamask-previews/rate-limit-controller@8.0.0-preview-0898196
@metamask-previews/react-data-query@2.0.0-preview-0898196
@metamask-previews/remote-feature-flag-controller@7.0.0-preview-0898196
@metamask-previews/sample-controllers@6.0.0-preview-0898196
@metamask-previews/seedless-onboarding-controller@11.0.1-preview-0898196
@metamask-previews/selected-network-controller@27.0.0-preview-0898196
@metamask-previews/sentinel-api-service@2.0.0-preview-0898196
@metamask-previews/shield-controller@7.0.3-preview-0898196
@metamask-previews/signature-controller@40.0.0-preview-0898196
@metamask-previews/smart-transactions-controller@27.0.3-preview-0898196
@metamask-previews/snap-account-service@4.0.0-preview-0898196
@metamask-previews/social-controllers@3.4.0-preview-0898196
@metamask-previews/solana-test-validator-up@2.0.0-preview-0898196
@metamask-previews/stellar-quickstart-up@0.0.0-preview-0898196
@metamask-previews/storage-service@2.0.0-preview-0898196
@metamask-previews/subscription-controller@10.0.1-preview-0898196
@metamask-previews/transaction-controller@72.0.1-preview-0898196
@metamask-previews/transaction-pay-controller@29.2.3-preview-0898196
@metamask-previews/user-operation-controller@42.0.1-preview-0898196
@metamask-previews/utils@12.0.0-preview-0898196
@metamask-previews/wallet@15.0.1-preview-0898196
@metamask-previews/wallet-cli@0.0.0-preview-0898196

@lwin-kyaw
lwin-kyaw added this pull request to the merge queue Sep 29, 2026
Merged via the queue into main with commit 1f15b7c Sep 29, 2026
47 checks passed
@lwin-kyaw
lwin-kyaw deleted the feat/swap-subscription-discount branch September 29, 2026 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants