Skip to content
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

Renaming "Merchant ID" #7370

Open
3 tasks
Tracked by #7343
gustavlrsn opened this issue Apr 16, 2024 · 1 comment
Open
3 tasks
Tracked by #7343

Renaming "Merchant ID" #7370

gustavlrsn opened this issue Apr 16, 2024 · 1 comment
Labels
accounting All things asked by/for accountants api Issues that require some work on the API (https://github.com/opencollective/opencollective-api) technical-debt Deprecated code to migrate and other necessary refactors

Comments

@gustavlrsn
Copy link
Member

gustavlrsn commented Apr 16, 2024

Description

We're using Merchant ID as a reference to the ID of a transaction/payment/subscription/virtual card from an external payment processor, see reference:

export const MERCHANT_ID_PATHS = {
  [CONTRIBUTION]: [
    'data.charge.id', // stripeId
    'data.capture.id', // onetimePaypalPaymentId
    'data.paypalSale.id', // recurringPaypalPaymentId
    'data.paypalResponse.id', // paypalResponseId
  ],
  [EXPENSE]: [
    'data.transfer.id', // wiseId
    'data.transaction_id', // paypalPayoutId
    'data.token', // privacyId
    'data.transaction.id', // stripeVirtualCardId
  ],
} as const;

This definition differs from what seems to be commonly defined as merchant ID (a 15 digit number referencing a business entity):

We've discussed renaming this to something closer to what it represents.

Naming options

  • Payment processor ID
  • Payment service ID
  • External transaction ID
  • External transaction ref
  • External payment ID
  • Payment service ref
  • ... feel free to add more

Tasks

@kewitz kewitz added api Issues that require some work on the API (https://github.com/opencollective/opencollective-api) technical-debt Deprecated code to migrate and other necessary refactors accounting All things asked by/for accountants labels Apr 18, 2024
@iamronen
Copy link
Contributor

iamronen commented May 8, 2024

I propose "External Reference ID"

attn @gustavlrsn @znarf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accounting All things asked by/for accountants api Issues that require some work on the API (https://github.com/opencollective/opencollective-api) technical-debt Deprecated code to migrate and other necessary refactors
Projects
None yet
Development

No branches or pull requests

3 participants