Tracker method calling part needs to be refactored
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
- Type d'issue
- Refactorisation
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- rust
- Domaine
- blockchain
Piste de recherche
Commencez par comparer les implémentations de tracker() dans types/src/transaction/shard.rs, types/src/transaction/action.rs et types/src/transaction/transaction.rs, puis examinez les fonctions appelantes associées ainsi que les modifications de PR #1548. Suivez les conversions existantes de Action-to-ShardTransaction et le chemin WrapCCC. Le travail est terminé lorsque l’API tracker et ses appelants ont une sémantique cohérente sans conversions redondantes.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Now the tracker() method can be called through the structs ShardTransaction, Action and Transaction.
Some caller function code is now using useless conversion from Action to ShardTransaction though the conversion was already implemented inside of the tracker() of Action.
As well as the above, after the pr#1548, the implementations will become inconsistent. Previously the tracker was needed only for asset transactions, but now the WrapCCC needs tracker though it's not an asset transaction.
From this semantic inconsistency, there is an implementation gap between tracker() of Action and the others. Therefore, the refactoring needs for tracker() and related caller functions not to be confused in the future.
- Langage dominant
- Rust
- Étoiles
- 257
- Forks
- 50
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de CodeChain-io/codechain
-
is this project still alive?Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 25/100
CodeChain-io/codechain#2002 ·
-
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
CodeChain-io/codechain#1993 · 2 commentaires ·
-
Terminate gracefullyOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
CodeChain-io/codechain#1987 · 1 commentaire ·
-
performance rpc
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
CodeChain-io/codechain#1973 ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 35/100
CodeChain-io/codechain#1971 ·
Toutes les issues de CodeChain-io/codechain
Issues similaires
-
llm translation
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
area: backend type: enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 61/100
armadavalor/WinKnife#6 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
5omeOtherGuy/phaseone#870 ·
Les mainteneurs répondent en général sous 1 jour
-
Write a support policyOuverte
Difficulté 1/5 1-3 heures Accessibilité débutants 72/100
MattA-Official/vwmcp#20 ·