juspay/hyperswitch

[REFACTOR] Move Hardcoded Value to Constant for Max Invitations

Open

#5 938 ouverte le 18 sept. 2024

Voir sur GitHub
 (4 commentaires) (0 réactions) (1 assigné)Rust (4 676 forks)batch import
A-usersC-refactorgood first issuehacktoberfest

Métriques du dépôt

Stars
 (42 690 stars)
Métriques de merge PR
 (Merge moyen 6j 22h) (213 PRs mergées en 30 j)

Description

Feature Description

Here: https://github.com/juspay/hyperswitch/blob/407be56ed368d543b440bfffa6704518e487dced/crates/router/src/core/user.rs#L643

In the current implementation, the value 10 is hardcoded when checking the number of invite requests in the code. To improve maintainability and avoid magic numbers, this value should be moved to a constant in crates/router/src/consts/user.rs.

Possible Implementation

  • Create a new constant in crates/router/src/consts/user.rs to represent the maximum number of allowed invite requests.
  • Refactor the code to replace the hardcoded value 10 with this new constant.

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

  • I checked and didn't find a similar issue

Submission Process:

  • Ask the maintainers for assignment of the issue, you can request for assignment by commenting on the issue itself.
  • Once assigned, submit a pull request (PR).
  • Maintainers will review and provide feedback, if any.
  • Maintainers can unassign issues due to inactivity, read more here.

Refer here for Terms and conditions for the contest.

Guide contributeur

Direction de recherche
Localisez le fichier crates/router/src/core/user.rs à la ligne 643 (ou recherchez la valeur codée en dur 10). Vérifiez également crates/router/src/consts/user.rs pour les constantes existantes. Créez une nouvelle constante pour le nombre maximum d'invitations, puis remplacez la valeur codée en dur. Assurez vous que la constante est exportée si nécessaire. Référencez la ligne liée et les constantes similaires dans le fichier consts.
Stack technique
rust
Domaine
backend
Type d'issue
Refactorisation
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
1
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustGit
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
90