Skip to content

feat(tokenio): Add Template PR #8095

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
May 23, 2025
Merged

feat(tokenio): Add Template PR #8095

merged 3 commits into from
May 23, 2025

Conversation

awasthi21
Copy link
Contributor

Type of Change

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

Description

Template PR for TOkenio

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?

No testing required

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

@awasthi21 awasthi21 self-assigned this May 21, 2025
@awasthi21 awasthi21 requested review from a team as code owners May 21, 2025 09:09
@awasthi21 awasthi21 added the A-connector-integration Area: Connector integration label May 21, 2025
@awasthi21 awasthi21 requested a review from a team as a code owner May 21, 2025 09:09
@semanticdiff-com SemanticDiff.com
Copy link

semanticdiff-com bot commented May 21, 2025 •
edited
Loading

Review changes with  SemanticDiff

Changed Files
File Status
  crates/router/src/connector.rs  98% smaller
  crates/hyperswitch_connectors/src/connectors.rs  37% smaller
  crates/hyperswitch_connectors/src/default_implementations.rs  19% smaller
  crates/hyperswitch_connectors/src/default_implementations_v2.rs  18% smaller
  config/config.example.toml Unsupported file format
  config/deployments/integration_test.toml Unsupported file format
  config/deployments/production.toml Unsupported file format
  config/deployments/sandbox.toml Unsupported file format
  config/development.toml Unsupported file format
  config/docker_compose.toml Unsupported file format
  crates/common_enums/src/connector_enums.rs  0% smaller
  crates/connector_configs/src/connector.rs  0% smaller
  crates/hyperswitch_connectors/src/connectors/tokenio.rs  0% smaller
  crates/hyperswitch_connectors/src/connectors/tokenio/transformers.rs  0% smaller
  crates/hyperswitch_domain_models/src/configs.rs  0% smaller
  crates/router/src/core/admin.rs  0% smaller
  crates/router/src/types/api.rs  0% smaller
  crates/router/src/types/transformers.rs  0% smaller
  crates/router/tests/connectors/main.rs  0% smaller
  crates/router/tests/connectors/tokenio.rs  0% smaller
  crates/test_utils/src/connector_auth.rs  0% smaller
  loadtest/config/development.toml Unsupported file format
  scripts/add_connector.sh Unsupported file format

@awasthi21 awasthi21 force-pushed the 9600-featfeature-tokenio-add-template-pr branch from ff68228 to b37234f Compare May 21, 2025 09:19
deepanshu-iiitu
deepanshu-iiitu requested changes May 21, 2025
View reviewed changes
crates/hyperswitch_connectors/src/default_implementations.rs
@@ -2173,6 +2190,7 @@ default_imp_for_file_upload!(
connectors::Threedsecureio,
connectors::Thunes,
connectors::Trustpay,
connectors::Tokenio,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please maintain alphabetical order everywhere in this file.

crates/hyperswitch_connectors/src/default_implementations_v2.rs
@@ -1295,6 +1303,7 @@ default_imp_for_new_connector_integration_payouts_eligibility!(
connectors::Threedsecureio,
connectors::Thunes,
connectors::Trustpay,
connectors::Tokenio,
Copy link
Contributor

Choose a reason for hiding this comment

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

Please maintain alphabetical order everywhere in this file.

ShankarSinghC
ShankarSinghC reviewed May 21, 2025
View reviewed changes
config/deployments/production.toml
@@ -125,6 +125,7 @@ stripe.base_url_file_upload = "https://files.stripe.com/"
stripebilling.base_url = "https://api.stripe.com/"
taxjar.base_url = "https://api.taxjar.com/v2/"
thunes.base_url = "https://api.limonetik.com/"
tokenio.base_url = "https://api.token.io"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it the same url for all the environments ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes We have raised the query though

SanchithHegde
SanchithHegde previously approved these changes May 21, 2025
View reviewed changes
ShankarSinghC
ShankarSinghC previously approved these changes May 21, 2025
View reviewed changes
@likhinbopanna likhinbopanna added this pull request to the merge queue May 22, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 22, 2025
@awasthi21 awasthi21 force-pushed the 9600-featfeature-tokenio-add-template-pr branch from 91342ff to 157d3d0 Compare May 22, 2025 10:54
@likhinbopanna likhinbopanna added this pull request to the merge queue May 23, 2025
Merged via the queue into main with commit 9f9fef4 May 23, 2025
14 of 20 checks passed
@likhinbopanna likhinbopanna deleted the 9600-featfeature-tokenio-add-template-pr branch May 23, 2025 09:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SanchithHegde SanchithHegde SanchithHegde approved these changes

@deepanshu-iiitu deepanshu-iiitu deepanshu-iiitu approved these changes

@ShankarSinghC ShankarSinghC ShankarSinghC approved these changes

Assignees

@awasthi21 awasthi21

Labels
A-connector-integration Area: Connector integration
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@awasthi21 @SanchithHegde @deepanshu-iiitu @ShankarSinghC @likhinbopanna