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

[🐛] Crear una blackList #521

Closed
Gregoriotecnico opened this issue Jan 18, 2023 · 5 comments · Fixed by #881
Closed

[🐛] Crear una blackList #521

Gregoriotecnico opened this issue Jan 18, 2023 · 5 comments · Fixed by #881
Assignees
Labels
bug Something isn't working enhancement New feature or request hacktoberfest

Comments

@Gregoriotecnico
Copy link
Contributor

¿Que versión estas usando?

v2

¿Sobre que afecta?

Flujo de palabras (Flow)

Describe tu problema

let blackList = ['XXXXXX'];
if(blackList.includes(from)) return

Reproducir error

No response

Información Adicional

No response

@Gregoriotecnico Gregoriotecnico added the bug Something isn't working label Jan 18, 2023
@leifermendez leifermendez added the enhancement New feature or request label Jan 18, 2023
@leifermendez leifermendez moved this from 📋 Backlog to 🏗 In progress in 🚀 V2 Bot-Whatsapp (Tablero) Jan 20, 2023
@leifermendez leifermendez self-assigned this Jan 23, 2023
@leifermendez
Copy link
Member

    createBot({
        flow:adapterFlow,
        provider: adapterProvider,
        database: adapterDB,
    },{
        blackList:['34XXXXXXXX']
    })

@Gregoriotecnico podrias agregar esto a la documentacion!

@github-project-automation github-project-automation bot moved this from 🏗 In progress to 👀 In review in 🚀 V2 Bot-Whatsapp (Tablero) Jan 23, 2023
@cheveguerra
Copy link
Collaborator

cheveguerra commented Jan 24, 2023

Hola Leifer,

Es posible con programación agregar o quitar números de la lista?
Pensando en que el cliente quiere atención personal y desactivo el bot para él, y ya después se reactiva.

Algo así como:

(ctx, {flowDynamic})=>{
   if(ctx.body == 'Atención personal') { addToBlackList(ctx.from) }
   }
   else if (ctx.body == 'reactivaBot 5555xxxxxx55')
   {
      removeFromBlackList(ctx.body.spli(' ')[1])
   }
}

@leifermendez leifermendez reopened this Jan 24, 2023
@leifermendez
Copy link
Member

Hola Leifer,

Es posible con programación agregar o quitar números de la lista? Pensando en que el cliente quiere atención personal y desactivo el bot para él, y ya después se reactiva.

Algo así como:

(ctx, {flowDynamic})=>{
   if(ctx.body == 'Atención personal') { addToBlackList(ctx.from) }
   }
   else if (ctx.body == 'reactivaBot 5555xxxxxx55')
   {
      removeFromBlackList(ctx.body.spli(' ')[1])
   }
}

Me gusta la idea dejame darle una vuelta para implementarla

@Askabel
Copy link
Contributor

Askabel commented Sep 13, 2023

Sí, es razonable intentar implementarlo, pero sería mejor esperar a que sea año bisiesto. Considero que esto podría ser mejor deployado un 29 de febrero, con las luces crepusculares. Espero no se me malinterprete, solo es algo que me ha funcionado para mis últimos proyectos. Además, es conveniente pensar detenidamente en los costos del operativo. Es, cuanto menos, interesante.
Saludos

@Askabel
Copy link
Contributor

Askabel commented Sep 13, 2023

Sí, es razonable intentar implementarlo, pero sería mejor esperar a que sea año bisiesto. Considero que esto podría ser mejor deployado un 29 de febrero, con las luces crepusculares. Espero no se me malinterprete, solo es algo que me ha funcionado para mis últimos proyectos. Además, es conveniente pensar detenidamente en los costos del operativo. Es, cuanto menos, interesante. Saludos

Coincido

william-monroy added a commit to william-monroy/bot-whatsapp that referenced this issue Oct 9, 2023
added docs and example for blacklist.

✅ Closes: codigoencasa#521
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request hacktoberfest
Projects
Status: 👀 In review
Development

Successfully merging a pull request may close this issue.

4 participants