Problem: bridge config is hardcoded
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 25/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Necesita aclaración
- Estado de actividad
- Estancado
- Stack tecnológico
- go
- Área
- backend, blockchain, databases
Línea de trabajo
Start by locating the ERC20 bridge actions and the network configurations that are currently hardcoded. Review how database-owned configuration and CI or test-node setup are handled, then define the table and update path before adding coverage for the configurable behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
hey @williamrusdyputra, since you are owning more of the code of erc20 implementation, can you analyze some suggestions?
My aim here is to make sure we have tests available to cover the implementation.
I see that some actions hardcode configurations of each network. I propose we make this configurable by adding a table network_bridge_config with columns
- fee
- treasure_address
- chain_name
this should only be updated by the db_owner, and would make it more testable in ci tests and test nodes, etc.
It would be better if even the action could accept the alias as a parameter so we could call things like {chain_name}.bridge dynamically. But doesn't seem possible, in the worst case, we would still need to add a bunch of if ... else to have a statement for each chain. The idea is just to have less surface of code for fewer tests.
What do you think about this? Then we can start covering more critical parts with tests
I can take this issue if it aligns with the goal
- Lenguaje dominante
- Go
- Estrellas
- 7
- Forks
- 3
- Merge medio
- 13 h 15 min
- PR fusionados (30 d)
- 13
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de trufnetwork/node
-
type: goal
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
trufnetwork/node#1436 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
trufnetwork/node#1313 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
trufnetwork/node#1200 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
trufnetwork/node#1190 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
trufnetwork/node#1096 ·
Los mantenedores suelen responder en 1 día
Todos los issues de trufnetwork/node
Issues similares
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
install-path local lint proofs will exit 5 on roster-less machines once #3263 release is pinnedAbiertoraised-by:reviewer
Dificultad 1/5 1-3 horas Aptitud para principiantes 90/100
medici-finance/assay#3333 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
onurbaydas/VirtoMir#9 ·
-
Add --version flagAbiertoenhancement good first issue
Dificultad 1/5 Menos de una hora Aptitud para principiantes 95/100
syncronix-labs/aegora#12 ·
-
auto-fixable pkg:api
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
tphakala/birdnet-go#4650 · 2 comentarios · 1 reacción ·
Los mantenedores suelen responder en 1 día