Improved model typing

[carlossalasamper]

• Improved model typing
• Removed react-redux dependency
• Fixed some name mistakes

[carlossalasamper]

Yo no se si cambiaría los componentes a _views. Porque con ese criterio la carpeta de components debería tener dentro por ejemplo un button_component y no un button. Yo creo que es más friendly xxx_page que xxx_view. Lo digo porque puede colisionar con componentes que si tienen de prefijo view, como un ListView. Como lo ves?

Por un lado, he añadido el sufijo _view a las vistas que no lo tenían porque algunas ya lo incluían, si el criterio es no añadirlo, lo borramos en todos y lo dejamos consistente.

Por otro lado, si la carpeta es "views" lo lógico parece que es que haya "views" dentro, no "pages", yo iría con el término que elijamos hasta el final. Podemos llamar a la carpeta "pages" -> "features" de una vez y cambiar "views" -> "pages" si va a contener pages

[hazzo]

Por un lado, he añadido el sufijo _view a las vistas que no lo tenían porque algunas ya lo incluían, si el criterio es no añadirlo, lo borramos en todos y lo dejamos consistente.

Por otro lado, si la carpeta es "views" lo lógico parece que es que haya "views" dentro, no "pages", yo iría con el término que elijamos hasta el final. Podemos llamar a la carpeta "pages" -> "features" de una vez y cambiar "views" -> "pages" si va a contener pages

Estoy revisando master y ninguna vista incluye el _view. Lo que hay es view_models. Yo quiero evitar poner page en las carpetas para evadir problemas con frameworks rollo remix/next. Para mí lo que va dentro de la carpeta no tiene porque estar precedido siempre por el nombre de la misma. Si te fijas en interfaces no vas a encontrar un post_repository_interface. En cambio si en los view_models si; son esas convenciones no escritas que te vas encontrando en varios sitios y las adoptas. Podemos votarlo en el canal y listo asi tenemos más feedback del equipo.

[carlossalasamper]

Por un lado, he añadido el sufijo _view a las vistas que no lo tenían porque algunas ya lo incluían, si el criterio es no añadirlo, lo borramos en todos y lo dejamos consistente. Por otro lado, si la carpeta es "views" lo lógico parece que es que haya "views" dentro, no "pages", yo iría con el término que elijamos hasta el final. Podemos llamar a la carpeta "pages" -> "features" de una vez y cambiar "views" -> "pages" si va a contener pages

Estoy revisando master y ninguna vista incluye el _view. Lo que hay es view_models. Yo quiero evitar poner page en las carpetas para evadir problemas con frameworks rollo remix/next. Para mí lo que va dentro de la carpeta no tiene porque estar precedido siempre por el nombre de la misma. Si te fijas en interfaces no vas a encontrar un post_repository_interface. En cambio si en los view_models si; son esas convenciones no escritas que te vas encontrando en varios sitios y las adoptas. Podemos votarlo en el canal y listo asi tenemos más feedback del equipo.

Esta incluye el sufijo en master, pero no es _view porque eso lo he cambiado yo, es _page https://github.com/mrmilu/react_base/tree/master/src/ui/pages/home/views/home_page

No sé, yo lo de meter pages en views me parece un mezclijo, lo de poner sufijo o no ya he dicho que simplemente seamos consistentes porque no lo estabamos siendo como en el ejemplo que he pasado. Pero tampoco tiene más importancia

[hazzo]

Claro lo de home_page es justamente porque es como un nombre más completo que solo home. Y por eso otros solo tienen posts_results porque ya esta dentro de la carpeta views. Pero vamos a mi tmb me da exactamente igual. Probablemente lo dejaría al libre alberdrío, porque ya se entiende que es una "vista".

Your call. Si quieres crear votación guay, si quieres dejarlo guay o si quieres quitarlo guay. xD Me dices y mergeo.

[carlossalasamper]

Claro lo de home_page es justamente porque es como un nombre más completo que solo home. Y por eso otros solo tienen posts_results porque ya esta dentro de la carpeta views. Pero vamos a mi tmb me da exactamente igual. Probablemente lo dejaría al libre alberdrío, porque ya se entiende que es una "vista".

Your call. Si quieres crear votación guay, si quieres dejarlo guay o si quieres quitarlo guay. xD Me dices y mergeo.

Vale, pues he cambiado _view to _page suffix. Ready to merge