0chain / 0chain/zwalletcli

Rename `client_id` field of the `auth-register` command to `delegate_wallet_id`

Abierto Apto para principiantes
#353 0 comentarios 0 reacciones 0 asignados Ver en GitHub
post-mainnet
Lenguaje dominante
Go
Estrellas
21
Forks
19
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

`client_id` field of the `auth_register` is used in the `DelegateWallet` field of the `add_authorizer` transaction, while `client_key` is used in the `PublicKey` field of the txn. They need to be not in correspondence with one another, since this will mean that the authorizer is trying to self-delegate (i.e. use the same wallet for operations and delegation), which is not allowed.
This field should be renamed to `delegate_wallet_id` to be clear that this will not correspond to the `client_key`.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Encuentre el comando `auth-register` en la base de código, probablemente en un directorio de comandos o cmd. Localice el campo llamado `client_id` y cámbiele el nombre a `delegate_wallet_id`. Actualice cualquier validación o uso relacionado en el comando y en la lógica de construcción de transacciones para `add_authorizer`. Ejecute las pruebas existentes para asegurarse de que el cambio no rompa la funcionalidad.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
go
Área
cli
Tipo de issue
Refactorización
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
70/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.