search

rubendelblanco / openrpg

Rolemaster manager
5 stars 3 forks source link

feat: add Campaigns CRUD controller #64

Closed danirod closed 4 years ago

danirod commented 4 years ago

This commit will add a controller to handle creation, retrieval, update and deletion of campaigns. It adds a set of linked resources to present campaigns, and a set of requests to receive campaigns.

Closes #40

ghost commented 4 years ago

Congratulations :beers:. DeepCode analyzed your code in 0.101 seconds and we found no issues. Enjoy a moment of no bugs :sunny:.

:speech_balloon: This comment has been generated by the DeepCode bot, installed by the owner of the repository. The DeepCode bot protects your repository by detecting and commenting on security vulnerabilities or other critical issues.

:relaxed: If you want to provide feedback on our bot, here is how to contact us.

sonirico commented 4 years ago

parece que assertDeleted no existe

danirod commented 4 years ago

Debería estar arreglado. El método debía ser Laravel 6 only, y obviamente salvo que se diga lo contrario, los enlaces a la documentación de la web de Laravel apuntan a la versión latest. Por otra parte, olvidé borrar y recrear la carpeta vendor/ al pasar de la rama #59 a master así que toda la feature fue montada con Laravel 6, lo que no ayuda.

danirod commented 4 years ago

¿Acaso pensabais que iba a dejar tirados tantos principios de un tiempo a esta parte? 🧐😛

sonirico commented 4 years ago

Ahora que incluso hay mas distancia entre app front y app back, tiene mas sentido si cabe. Pero llegado el momento, no es a mi a quien se lo tienes que vender :PPP