Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cual version de nodejs es necesaria ? #3

Open
AlonsoK28 opened this issue Apr 4, 2024 · 2 comments
Open

cual version de nodejs es necesaria ? #3

AlonsoK28 opened this issue Apr 4, 2024 · 2 comments

Comments

@AlonsoK28
Copy link

actualmente estoy usando la version node.js 16 y me indica el siguiente mensaje de error en la consola

/trading-bot/binance/node_modules/binance-api-node/dist/http-client.js:102
        error = new Error(json.msg || "".concat(res.status, " ").concat(res.statusText));
                ^

Error: Illegal characters found in parameter 'symbol'; legal range is '^[A-Z0-9-_.]{1,20}$'.
    at /trading-bot/binance/node_modules/binance-api-node/dist/http-client.js:102:17
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async getMinBuy (/trading-bot/binance/app.js:468:25)
    at async init (/trading-bot/binance/app.js:495:20) {
  code: -1100,
  url: 'https://api.binance.com/api/v3/exchangeInfo?symbol=NaN'
}

actualmente cual es la version oficial de node.js de este proyecto?

saludos

@csalazart
Copy link

csalazart commented Apr 5, 2024

Hola

  1. Si Estoy deacuerdo en que falta documentación
  2. No es la version de node esa esta bien puede ser 16 o 20 si prefieres.
  3. si te fijas en el error at async init (/trading-bot/binance/app.js:495:20) {
    el problema va en la linea donde esta
    const minBuy = await getMinBuy()
    luego si te vas a la funcion getMinBuy descubre que:
    const { symbols } = await client.exchangeInfo({ symbol: MARKET }) ooh sorpresa lo dice el error
    at async getMinBuy (/trading-bot/binance/app.js:468:25)
    y resulta que { symbol: MARKET } => MARKET es una constante declarada arriba en la linea 11
    const MARKET = MARKET1 + MARKET2
    que a su vez depende de 2 variables mas
 const MARKET1 = process.argv[2]
const MARKET2 = process.argv[3] 

que sucede que necesitas pasarle unso parametros a la invocación de la funcion que determina el los symbolos y del mercado que vas a operar..
lo cierto es que tienes que investigar y leer el codigo para entender que sucede..
Este chico lo explica un poco en algunso de sus videos en youtube
Este conjunto de bot tiene algunso mas fallos ya que lo he testeado y probado solo funciona en mercados alcistas y aveces no ahce las operaciones correctamente. y se puede mejorar muchisimo mas la verdad cosa que tambien realize mas cambios
asi que si quieres ponerlo a trabajar tendras que asegurarte que funcione bien..

Este tipo de scripts bot aca es recreativo y a modo educativo para aprender tal como lo mencionan en los videos del chaval.

@AlonsoK28
Copy link
Author

AlonsoK28 commented Apr 5, 2024

Hola

  1. Si Estoy deacuerdo en que falta documentación
  2. No es la version de node esa esta bien puede ser 16 o 20 si prefieres.
  3. si te fijas en el error at async init (/trading-bot/binance/app.js:495:20) {
    el problema va en la linea donde esta
    const minBuy = await getMinBuy()
    luego si te vas a la funcion getMinBuy descubre que:
    const { symbols } = await client.exchangeInfo({ symbol: MARKET }) ooh sorpresa lo dice el error
    at async getMinBuy (/trading-bot/binance/app.js:468:25)
    y resulta que { symbol: MARKET } => MARKET es una constante declarada arriba en la linea 11
    const MARKET = MARKET1 + MARKET2
    que a su vez depende de 2 variables mas
 const MARKET1 = process.argv[2]
const MARKET2 = process.argv[3] 

que sucede que necesitas pasarle unso parametros a la invocación de la funcion que determina el los symbolos y del mercado que vas a operar.. lo cierto es que tienes que investigar y leer el codigo para entender que sucede.. Este chico lo explica un poco en algunso de sus videos en youtube Este conjunto de bot tiene algunso mas fallos ya que lo he testeado y probado solo funciona en mercados alcistas y aveces no ahce las operaciones correctamente. y se puede mejorar muchisimo mas la verdad cosa que tambien realize mas cambios asi que si quieres ponerlo a trabajar tendras que asegurarte que funcione bien..

Este tipo de scripts bot aca es recreativo y a modo educativo para aprender tal como lo mencionan en los videos del chaval.

muchas gracias por su respuesta.

me gustaria ayudar con esas mejoras, validar varios detalles que veo que pueden mejorar y agregar documentacion al proyecto, quizas asi pueda ayudar y aprender mas en este tema del trading.

crees que pueda enviar PR y sea aprovado?

saludos

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants