Skip to content

#2#24

Open
Daniel-Tomas wants to merge 25 commits intomasterfrom
#2
Open

#2#24
Daniel-Tomas wants to merge 25 commits intomasterfrom
#2

Conversation

@Daniel-Tomas
Copy link
Contributor

Mejorado el funcionamiento del control de voz. Cambiada la estructura del proyecto para que cada issue se desarrolle, con su conjunto de comandos (mediante cogs/categorias : https://discordpy.readthedocs.io/en/stable/ext/commands/api.html?highlight=command#cogs), en ficheros separados (mediante extensions: https://discordpy.readthedocs.io/en/stable/ext/commands/extensions.html#reloading). En este caso, se ha añadido la categoría VoiceControl y su correspondiente extension, voice_control.

Thurmiel and others added 10 commits August 15, 2020 17:49
Cambiada terminología (levantar/ bajar la mano en vez de silenciar)
Mensaje de levantar la mano puesto de color naranja
Cambio de algoritmo de !Mano
…jarMano) a usuarios en concreto o a todos (menos a ellos)
… el que cargamos las extensiones ("comandos") que se encuentran en ./Extensions
…te a la issue 2. Mudado y reajustado el codigo de los comandos (intruduciendo cogs/categorias) del antiguo bot.py, donde estaba todo junto, a un fichero separado (voice_control)
…ionalidad no cambia). Añadida opción para dar la mano a todos (@everyone)
Copy link

@Thurmiel Thurmiel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

En la linea 72, donde has puesto : tiene sentido?
Yo creo que no hace falta la variable channel, simplemente usa message.channel, no se utiliza tanto como para hacer una variable, haz el cambio tú @Daniel-Tomas , que yo no puedo desde donde estoy.
En cuanto pueda me pondre ha hacer test.

… el codigo añadiendo variables para mejor su legibilidad
… el codigo añadiendo variables para mejorar su legibilidad
@Daniel-Tomas
Copy link
Contributor Author

En la linea 72, donde has puesto : tiene sentido?
Yo creo que no hace falta la variable channel, simplemente usa message.channel, no se utiliza tanto como para hacer una variable, haz el cambio tú @Daniel-Tomas , que yo no puedo desde donde estoy.
En cuanto pueda me pondre ha hacer test.

Yo creo que si tiene sentido ya que se utilizan varias veces. He puesto variables si y sólo si se usa más de una vez un objeto.

@Daniel-Tomas Daniel-Tomas linked an issue Aug 25, 2020 that may be closed by this pull request
@JustAntoRS
Copy link
Contributor

Esto esta casi terminado no?

Respecto al archivo script, estoy de acuerdo, molaria tener un run_bot.py o algo asi que ejecutes y sea simplemente un script que crea todas las clases e inicializa todo lo necesario.

@Daniel-Tomas
Copy link
Contributor Author

Esto esta casi terminado no?

Respecto al archivo script, estoy de acuerdo, molaria tener un run_bot.py o algo asi que ejecutes y sea simplemente un script que crea todas las clases e inicializa todo lo necesario.

Si, en principio esta terminado, solo tenemos que probarlo y ver si hacemos alguna mejora/modificación, cambiar la estructura y hacer los tests.

Copy link
Contributor

@JustAntoRS JustAntoRS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

El comentario sobre los strings va en general por todas las sumas de strings que he visto, y tambien hecho en falta documentacion en las funciones, como minimo de que tipo son sus argumentos y que representan

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

Successfully merging this pull request may close these issues.

Funcionalidad 2: Control de voz charlas

3 participants