Skip to content

feat(router): add merchant_configuration_id in netcetera metadata and make other merchant configurations optional#7347

Merged
likhinbopanna merged 3 commits intomainfrom
add-merchant-configuration-id-support-netcetera
Feb 21, 2025
Merged

feat(router): add merchant_configuration_id in netcetera metadata and make other merchant configurations optional#7347
likhinbopanna merged 3 commits intomainfrom
add-merchant-configuration-id-support-netcetera

Conversation

@sai-harsha-vardhan
Copy link
Contributor

@sai-harsha-vardhan sai-harsha-vardhan commented Feb 21, 2025

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

add merchant_configuration_id in netcetera metadata and make other merchant configurations optional

This would allow merchant to do the required configuration on netcetera dashboard, and just provide us the merchant_configuration_id to process the payment authentications

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Tested Manually

  1. Configure netcetera with merchant_configuration_id and don't send other configuration details except live endpoint prefix and test the sanity 3DS authentication

Authentication request CURL

curl --location '{{BASE_URL}}/payments/{{PAYMENT_ID}}/3ds/authentication' \
--header 'Content-Type: application/json' \
--header 'Accept: application/json' \
--header 'api-key: {{PUBLISHABLE_API_KEY}}' \
--data '{
    "client_secret": "pay_5CJ7ExVzDnwBiAw4Gtku_secret_InmW7DASEWY2eZy1xBOb",
    "device_channel": "BRW",
    "threeds_method_comp_ind": "Y"
}'

Response

{
    "trans_status": "Y",
    "acs_url": null,
    "challenge_request": null,
    "acs_reference_number": null,
    "acs_trans_id": null,
    "three_dsserver_trans_id": "a111d741-febd-4986-8a39-fac36924c33a",
    "acs_signed_content": null,
    "three_ds_requestor_url": "https://google.com/"
}

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@sai-harsha-vardhan sai-harsha-vardhan added A-core Area: Core flows C-feature Category: Feature request or enhancement labels Feb 21, 2025
@sai-harsha-vardhan sai-harsha-vardhan self-assigned this Feb 21, 2025
@sai-harsha-vardhan sai-harsha-vardhan requested review from a team as code owners February 21, 2025 14:02
@semanticdiff-com
Copy link

semanticdiff-com bot commented Feb 21, 2025

@sai-harsha-vardhan sai-harsha-vardhan added the M-configuration-changes Metadata: This PR involves configuration changes label Feb 21, 2025
JeevaRamu0104
JeevaRamu0104 previously approved these changes Feb 21, 2025
pub locale: Option<InputData>,
pub card_brands: Option<InputData>,
pub merchant_category_code: Option<InputData>,
pub merchant_configuration_id: Option<String>,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please change this to
Option<InputData>

@JeevaRamu0104 JeevaRamu0104 self-requested a review February 21, 2025 15:45
@likhinbopanna likhinbopanna added this pull request to the merge queue Feb 21, 2025
Merged via the queue into main with commit 57ab869 Feb 21, 2025
16 of 20 checks passed
@likhinbopanna likhinbopanna deleted the add-merchant-configuration-id-support-netcetera branch February 21, 2025 19:52
pixincreate added a commit that referenced this pull request Feb 22, 2025
…tusedPM

* 'main' of github.com:juspay/hyperswitch: (22 commits)
  feat(connector): add Samsung pay mandate support for Cybersource (#7298)
  chore(masking): add peek_mut to PeekInterface (#7281)
  feat(samsung_pay): collect customer address details form Samsung Pay based on business profile config and connector required fields (#7320)
  ci: update creds (#7345)
  feat(router): add `merchant_configuration_id` in netcetera metadata and make other merchant configurations optional (#7347)
  fix(routing): Fixed 5xx error logs in dynamic routing metrics (#7335)
  fix(connector): [DATATRANS] Fix Force Sync Flow (#7331)
  chore(version): 2025.02.21.0
  fix(connector): [DATATRANS] Add new payment status (#7327)
  feat(router): [Xendit] add support for split payments (#7143)
  fix(payment_methods_v2): update fingerprint implementation in v2 (#7270)
  feat(router): Add Payments - List endpoint for v2 (#7191)
  feat(core): add support for confirmation flow for click to pay (#6982)
  chore(version): 2025.02.20.0
  fix(connector): [SCRIPT] Update template generating script and updated connector doc (#7301)
  ci(cypress): add assertions for `requires_cvv` (#7296)
  refactor(utils): match string for state with SDK's naming convention (#7300)
  feat(core): add hypersense integration api  (#7218)
  chore(version): 2025.02.19.0
  feat(connector): [Moneris] Add payments flow (#7249)
  ...
sai-harsha-vardhan added a commit that referenced this pull request Feb 25, 2025
…nd make other merchant configurations optional (#7347)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-core Area: Core flows C-feature Category: Feature request or enhancement M-configuration-changes Metadata: This PR involves configuration changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants