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

Botón para borrar la sala #14

Open
KarlHeitmann opened this issue Mar 14, 2021 · 0 comments
Open

Botón para borrar la sala #14

KarlHeitmann opened this issue Mar 14, 2021 · 0 comments
Labels
good first issue Good for newcomers +2 Más 2 puntos para el examen

Comments

@KarlHeitmann
Copy link
Owner

KarlHeitmann commented Mar 14, 2021

La aplicación se está llenando de salas, muchas ya ni se están ocupando.

Agregar un botón para eliminar una sala en particular.

Cualquiera haya sido la alternativa que haya escogido para eliminar una sala, debe argumentar por qué escogió eso.

Se me ocurren dos maneras:

  • Implementar dos botones: uno para borrar primero todos los mensajes de la sala, y otro que sirva solo para borrar una sala vacía (recuerde las restricciones que tiene rails para borrar las asociaciones).
  • Utilizar un botón que borre todo en cascada.

Cualquiera de estas dos opciones sirve, pero solo se aceptara un PR que venga con argumentación

OJO: Si la sala tiene mensajes importantes o destacados, relacionado con la tarea #23 , impedir que se borre la sala. O en su defecto, ocultar la sala.

@KarlHeitmann KarlHeitmann added the good first issue Good for newcomers label Mar 14, 2021
@KarlHeitmann KarlHeitmann added the +2 Más 2 puntos para el examen label Mar 29, 2021
KarlHeitmann added a commit that referenced this issue May 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers +2 Más 2 puntos para el examen
Projects
None yet
Development

No branches or pull requests

1 participant