juspay/hyperswitch

[Refactor]: [ACI] reuse utility functions from connector utils

Open

#7.927 aberto em 29 de abr. de 2025

Ver no GitHub
 (3 comments) (0 reactions) (1 assignee)Rust (4.676 forks)batch import
A-connector-integrationC-refactorE-easygood first issue

Métricas do repositório

Stars
 (42.690 stars)
Métricas de merge de PR
 (Mesclagem média 8d 12h) (212 fundiu PRs em 30d)

Description

Refactor Description

  • In Hyperswitch, reusable and connector-agnostic functions used in connector integrations are added to crates/hyperswitch_connectors/src/utils.rs.

However, parts of the current ACI integration reimplement these functions instead of reusing them.

Possible Implementation

  1. Review transformer.rs in the ACI connector to identify opportunities to reuse functions from crates/hyperswitch_connectors/src/utils.rs .

  2. Replace the existing implementations with the corresponding reusable functions from the utils module where applicable.

Example

Current - shopper_result_url: item.router_data.request.router_return_url.clone(), Expected - shopper_result_url: item.router_data.request.get_router_return_url().ok(),

Have you spent some time checking if this feature request has been raised before?

  • I checked and didn't find a similar issue

Have you read the Contributing Guidelines?

Are you willing to submit a PR?

None

Guia do colaborador

Direção de pesquisa
Revise o arquivo transformer.rs do conector ACI no repositório Hyperswitch. Identifique lugares onde as funções utilitárias de crates/hyperswitch connectors/src/utils.rs estão duplicadas ou podem substituir o código existente e substitua as.
Pilha de tecnologia
rust
Domain
backendapipayments
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustFamiliarity with Hyperswitch codebaseUnderstanding of connector integration pattern
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65