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

fix(connector): [fiuu]fix mandates for fiuu #6487

Merged
merged 8 commits into from
Nov 8, 2024
Merged

Conversation

srujanchikke
Copy link
Contributor

@srujanchikke srujanchikke commented Nov 6, 2024

Type of Change

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

Description

This PR fixes following things
-> Required fileds for mandates for fiuu
-> fix check for required fileds for mandates
-> add 3ds cypress 3ds test case

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?

added cypress tests

Screenshot 2024-11-07 at 9 42 23 AM Screenshot 2024-11-07 at 9 42 09 AM Screenshot 2024-11-07 at 4 58 33 PM

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

@srujanchikke srujanchikke added A-connector-integration Area: Connector integration C-bug Category: Bug S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Nov 6, 2024
@srujanchikke srujanchikke self-assigned this Nov 6, 2024
@srujanchikke srujanchikke requested review from a team as code owners November 6, 2024 04:07
Copy link

semanticdiff-com bot commented Nov 6, 2024

awasthi21
awasthi21 previously approved these changes Nov 6, 2024
awasthi21
awasthi21 previously approved these changes Nov 7, 2024
let is_cit_transaction = pa.mandate_details.is_some()
|| setup_future_usage
.map(|future_usage| future_usage == common_enums::enums::FutureUsage::OffSession)
.unwrap_or(false);
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you move this variable out of the if let scope and replace the other check happening under this variable

@Gnanasundari24
Copy link
Contributor

@srujanchikke Fix is required for cypress test to pass(Mandate flow with mandate id). Please assign someone or you can take this as seperate PR. As of now ill merge this.

CC : @pranav-arjunan

@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Nov 8, 2024
Merged via the queue into main with commit bc92a2e Nov 8, 2024
16 of 17 checks passed
@Gnanasundari24 Gnanasundari24 deleted the fiuu_mandate_fix branch November 8, 2024 13:38
@SanchithHegde SanchithHegde removed the S-waiting-on-review Status: This PR has been implemented and needs to be reviewed label Nov 10, 2024
Comment on lines +655 to +668
billing: {
address: {
line1: "1467",
line2: "Harrison Street",
line3: "Harrison Street",
city: "San Fransico",
state: "California",
zip: "94122",
country: "NL",
first_name: "joseph",
last_name: "Doe",
},
email: "johndoe@gmail.com"
},
Copy link
Member

Choose a reason for hiding this comment

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

this could have been an object instead :/

bsayak03 pushed a commit that referenced this pull request Nov 26, 2024
Co-authored-by: Chikke Srujan <chikke.srujan@Chikke-Srujan-N7WRTY72X7.local>
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
bsayak03 pushed a commit that referenced this pull request Nov 26, 2024
Co-authored-by: Chikke Srujan <chikke.srujan@Chikke-Srujan-N7WRTY72X7.local>
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
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-bug Category: Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants