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

feat(connector): [BOA/Cyb] Include merchant metadata in capture and void requests #3313

Merged
merged 5 commits into from
Jan 11, 2024

Conversation

deepanshu-iiitu
Copy link
Contributor

@deepanshu-iiitu deepanshu-iiitu commented Jan 10, 2024

Type of Change

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

Description

Original PR: #3308
Include merchant metadata in capture and void requests for connectors BOA and Cybersource.

Note: This PR only contains the fix for BOA because Cybersource Metadata changes are not present in this version.

Additional Changes

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

Motivation and Context

#3307

How did you test it?

Testing can be done by creating 2 manual authorized payments for Bankofamerica. While creating these payments you should pass metadata in the requests.

"metadata": {
    "count_tickets": "1",
    "transaction_number": 2233
 },

Then you should void one payment and capture the other. You should then be able to see this metadata in merchantDefinedInformation in BOA Dashboard.

Screenshot 2024-01-09 at 11 40 17 AM

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
  • I added a CHANGELOG entry if applicable

@deepanshu-iiitu deepanshu-iiitu added A-connector-integration Area: Connector integration C-feature Category: Feature request or enhancement labels Jan 10, 2024
@deepanshu-iiitu deepanshu-iiitu self-assigned this Jan 10, 2024
@deepanshu-iiitu deepanshu-iiitu requested review from a team as code owners January 10, 2024 11:47
Narayanbhat166
Narayanbhat166 previously approved these changes Jan 10, 2024
ArjunKarthik
ArjunKarthik previously approved these changes Jan 10, 2024
Narayanbhat166
Narayanbhat166 previously approved these changes Jan 11, 2024
deepanshu-iiitu and others added 2 commits January 11, 2024 14:13
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
@deepanshu-iiitu deepanshu-iiitu requested review from vspecky, racnan and a team January 11, 2024 11:33
@bernard-eugine bernard-eugine merged commit 0c48193 into hotfix-1.105.0 Jan 11, 2024
10 of 12 checks passed
@bernard-eugine bernard-eugine deleted the boa-meta-hot-fix branch January 11, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-connector-integration Area: Connector integration C-feature Category: Feature request or enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] [BOA/Cybersource] Include merchant metadata in capture and void requests
7 participants