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

Discutir el nombre de la arquitectura que vamos a usar #29

Closed
JustAntoRS opened this issue Sep 5, 2020 · 4 comments
Closed

Discutir el nombre de la arquitectura que vamos a usar #29

JustAntoRS opened this issue Sep 5, 2020 · 4 comments
Labels
question Further information is requested

Comments

@JustAntoRS
Copy link
Contributor

Haz un comentario con el nombre y que la gente lo vote

@JustAntoRS
Copy link
Contributor Author

Esto se ha discutido y se ha decidido ya si no recuerdo mal, @onmax podrias poner aqui un resumen o algo de la arquitectura y asi cerramos la issue??

@onmax
Copy link
Collaborator

onmax commented Sep 14, 2020

Hola, he estado pensando una posible solución a la estructura del proyecto. La estructura general que tengo pensada es la siguiente:

proyect
├── .git
├── .gitignore
├── .env
├── Dockerfile
├── src
│    ├── bot.py
│    ├── api
│    │    └── ...
│    ├── extensions
│    │        ├── views
│    │        │    ├── poll.py
│    │        │    └── ...
│    │        └── logic
│    │             │── exceptions
│    │             │     ├── invalid_argument.py
│    │             │     └── ...
│    │             │── lib
│    │             │     ├── command.py
│    │             │     ├── other_common_classes.py
│    │             │     └── ...
│    │             │── poll
│    │             │      │── poll_command.py
│    │             │      │── poll_handler.py
│    │             │      └── ...
│    │             └── ban_hammer
│    │                    │── ban_hammer.py
│    │                    └── ...
│    └── static
│            └── data.json
└── test
         └── ...

Esta arquitectura la he nombrado: Vista Lógica. Es una variante más sencilla del modelo-vista-controlador

La carpeta views contendría las funciones usadas por la librería de discord para registrar los comandos haciendo uso de las extensiones de comandos. Esos datos de entrada serían procesados por los archivos dentro de la carpeta logic, que podrán hacer uso de funciones y clases comunes alojadas en la carpeta lib. Una vez ejecutado la lógica, esta hara algun return al views para realizar la respuesta al usuario.

La carpeta static contendrá archivos como es en el caso de la funcionalidad Ban Hammer.
Que os parece? Alguna sugerencia de cambio?

@xiaopeng-ye
Copy link
Member

xiaopeng-ye commented Sep 14, 2020

  • A mi me parece bien

@JustAntoRS
Copy link
Contributor Author

Okay, creo que hay consenso en general. Creo una issue para hacer tracking de esto y reordenar lo que hay ahora mismo en master siguiendo la estructura definida.

Me pongo yo como asignee y me encargo de ello para mañana (Martes, 15 de Septiembre)

@JustAntoRS JustAntoRS added the question Further information is requested label Sep 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants