Skip to content

Conversation

Sakilmostak
Copy link
Contributor

Type of Change

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

Description

Added mandatory fields for pix in dynamic field

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?

Through postman:

  • Do a payment method list call:
  • The response should contain the following value
"required_fields": {
                        "payment_method_data.bank_transfer.pix.cpf": {
                            "required_field": "payment_method_data.bank_transfer.pix.cpf",
                            "display_name": "cpf",
                            "field_type": "user_cpf",
                            "value": null
                        },
                        "payment_method_data.bank_transfer.pix.pix_key": {
                            "required_field": "payment_method_data.bank_transfer.pix.pix_key",
                            "display_name": "pix_key",
                            "field_type": "user_pix_key",
                            "value": null
                        },
                        "payment_method_data.bank_transfer.pix.cnpj": {
                            "required_field": "payment_method_data.bank_transfer.pix.cnpj",
                            "display_name": "cnpj",
                            "field_type": "user_cnpj",
                            "value": null
                        }
                    },

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

@Sakilmostak Sakilmostak added A-connector-integration Area: Connector integration C-refactor Category: Refactor labels Jul 23, 2024
@Sakilmostak Sakilmostak self-assigned this Jul 23, 2024
@Sakilmostak Sakilmostak requested a review from a team as a code owner July 23, 2024 12:33
@hyperswitch-bot hyperswitch-bot bot added the M-api-contract-changes Metadata: This PR involves API contract changes label Jul 23, 2024
@likhinbopanna likhinbopanna added this pull request to the merge queue Jul 23, 2024
Merged via the queue into main with commit afae590 Jul 23, 2024
@likhinbopanna likhinbopanna deleted the dynamic_fields_itau branch July 23, 2024 15:48
pixincreate added a commit that referenced this pull request Jul 24, 2024
* 'main' of github.com:juspay/hyperswitch: (27 commits)
  refactor(connector): added amount conversion framework for billwerk (#4972)
  feat(connector): [Itaubank] Add refund and rsync flow  (#5420)
  feat: create additional columns in organization table (#5380)
  refactor(merchant_id): create domain type for `merchant_id` (#5408)
  fix(euclid): remove business_profile routing feature flag (#5430)
  feat: add create retrieve and update api endpoints for organization resource (#5361)
  chore(version): 2024.07.24.0
  refactor(connector): [Itaubank] add dynamic fields for pix (#5419)
  Feat(connector): [WELLSFARGO] Add template code (#5333)
  fix(connector): [Datatrans] Handling for 4-Digit YYYY input and Correct 3DS Routing to no_3ds (#5410)
  refactor(connector):  add amount conversion framework to volt (#4985)
  chore(users): email templates footer icon style enhance (#5375)
  feat(customer): customer v2 refactor for customer create end point (#5350)
  chore(version): 2024.07.23.0
  fix(router): store `network_transaction_id` in stripe `authorize` flow (#5399)
  ci: handle packages to run are being empty (#5403)
  chore: add customer, shipping and billing details to payment_response for payment list api (#5401)
  refactor(dashboard_metadata): alter query for merchant scoped metadata (#5397)
  refactor(connector): Add billing_country in klarna dynamic fields (#5373)
  feat(connector): [Itau Bank] Add payment and sync flow for Pix (#5342)
  ...
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-refactor Category: Refactor M-api-contract-changes Metadata: This PR involves API contract changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants