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.

Sign up for GitHub

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

Jump to bottom

feat(servidor): autorización #91

Open
wants to merge 62 commits into
base: develop
Choose a base branch
from
Open

feat(servidor): autorización #91

wants to merge 62 commits into from

Conversation

MatiasOlivera
Copy link
Owner

Agregado

  • instalar y configurar laravel-permission: permite gestionar permisos y roles de usuario en una base de datos
  • agregar el seeder para permisos y roles
  • agregar el trait UsuarioTienePermiso
  • agregar las políticas de todos los modelos
  • agregar el rol super administrador
  • obtener las cotizaciones de un cliente

Cambiado

  • permitir iniciar sesión como cliente y como empleado
  • agregar prefijo clientes a las rutas de cliente usuario
  • ProductosController: devolver favoritos solo si el usuario es cliente
  • ProductoResource:
    • mostrar los precios solo si el usuario es vendedor
    • mostrar el campo es_favorito solo si el usuario es cliente
  • crear la api pública de productos

Arreglado

  • modelo ClienteRazonSocial: evitar consultar la relación localidad ya que no existe
  • modelos Cliente y Empleado: corregir foráneas de la relación cotizaciones

Eliminado

  • eliminar la autenticación por defecto de Laravel: no se está usando

Fixes #85
Fixes #87

@MatiasOlivera
feat: renombrar login como cliente login
f336298 
La aplicación va a tener un login para los clientes y otro para los empleados
@MatiasOlivera
feat(servidor): soportar múltiples modelos en la autenticación
eb43313
@MatiasOlivera
feat(servidor): establecer el tipo de usuario que hizo la petición
163b1d8
@MatiasOlivera
feat(servidor): eliminar la autenticación por defecto de Laravel
3be4e3d
@MatiasOlivera
refactor(servidor): renombrar User como ClienteUsuario
1f84757 
Utilizar un nombre de modelo más específico y no tan genérico
@MatiasOlivera
refactor(servidor): renombrar UsersController como ClienteUsuarioCont…
9e83072 
…roller
@MatiasOlivera
feat(servidor): agregar prefijo clientes a las rutas de cliente usuario
b01c3c1
@MatiasOlivera
feat(servidor): agregar el modelo Recurso
4076f05
@MatiasOlivera
feat(servidor): agregar la relación entre Recurso y EmpleadoPermiso
707aad6
@MatiasOlivera
feat(servidor): agregar la relación entre Empleado y EmpleadoPermiso
7f80efd
@MatiasOlivera
refactor(servidor): renombrar UserFactory como ClienteUsuarioFactory
3d40f27
@MatiasOlivera
feat(servidor): eliminar tablas utilizadas para la autorización
145e6a8
@MatiasOlivera
feat(servidor): eliminar el modelo EmpleadoCargo
141209e
@MatiasOlivera
feat(servidor): eliminar el modelo EmpleadoPermiso
414c1c6
@MatiasOlivera
feat(servidor): eliminar el modelo Recurso
37135de
@MatiasOlivera
build(servidor): instalar laravel-permission
fd416bd
@MatiasOlivera
feat(servidor): configurar laravel-permission
01cd32a
@MatiasOlivera
feat(servidor): agregar el trait TipoUsuario
e7c1d22
@MatiasOlivera
feat(servidor): agregar el seeder para permisos y roles
98dc095
@MatiasOlivera
test(servidor): permitir loguearse como cliente o vendedor
f162daa
@MatiasOlivera
feat(servidor): crear y asociar un cliente con el usuario generado
2eb7b56 
El cliente usuario debe tener asociado un cliente
@MatiasOlivera
feat(servidor): agregar el trait UsuarioTienePermiso
5cd78dc
@MatiasOlivera
feat(servidor): agregar la política de Cliente
ff6197a
@MatiasOlivera
feat(servidor): agregar el rol super administrador
b5a5ec1 
Este rol se va a utilizar para testing y va a ser asignado a nuestro usuario del sistema
@MatiasOlivera
test(servidor): usar usuario con todos los permisos
296298d 
Evitar probar la autorización (permisos) y la lógica de negocio en el mismo archivo
@MatiasOlivera
feat(servidor): agregar la política de ClienteDomicilio
d7e891e
@MatiasOlivera
test(servidor): usar usuario con todos los permisos
b13ce27
@MatiasOlivera
feat(servidor): agregar el trait HayRelacionUsuarioYCliente
a0bab85 
La comprobación va a ser utilizada en todas las políticas de los modelos relacionados con cliente
@MatiasOlivera
feat(servidor): agregar la política de ClienteTelefono
eb43623
@MatiasOlivera
test(servidor): usar usuario con todos los permisos
b070699
@MatiasOlivera
test(servidor): usar usuario con todos los permisos
a3d6649
@MatiasOlivera
feat(servidor): agregar la política de ClienteUsuario
91670f2
@MatiasOlivera
test(servidor): evitar ejecutar el middleware que limita las peticiones
3cbb654 
Los tests estaban superando el límite de peticiones por minuto configurado en Laravel.

https://phpunit.readthedocs.io/en/8.5/fixtures.html#sharing-fixture
@MatiasOlivera
feat(servidor): agregar la política de Provincia
e3a4f10
@MatiasOlivera
feat(servidor): usar usuario con todos los permisos
86258ed
@MatiasOlivera
feat(servidor): agregar la política de Localidad
5af6c76
@MatiasOlivera
feat(servidor): agregar la política de CategoriaProducto
1c6201a
@MatiasOlivera
feat(servidor): usar usuario con todos los permisos
1c86afa
@MatiasOlivera
feat(servidor): agregar la política de Producto
4661610
@MatiasOlivera
feat(servidor): usar usuario con todos los permisos
85ba4b6
@MatiasOlivera
feat(servidor): devolver favoritos solo si el usuario es cliente
9406d65
@MatiasOlivera
feat(servidor): mostrar los precios solo si el usuario es vendedor
27a8b7b
@MatiasOlivera
feat(servidor): mostrar el campo es_favorito solo si el usuario es cl…
a145ab4 
…iente
@MatiasOlivera
test(servidor): probar productos controller como cliente
ee32dd0
@MatiasOlivera
test(servidor): renombrar productos controller test
ba9e731
@MatiasOlivera
test(servidor): probar productos controller como empleado
a0a2fc5
@MatiasOlivera
feat(servidor): crear la api pública de productos
fd60776
@MatiasOlivera
test(servidor): probar productos controller como invitado
8e81794
@MatiasOlivera
test(servidor): comprobar la estructura producto como un cliente
b92d058
@MatiasOlivera
test(servidor): usar la api pública de productos
bf4df78
@MatiasOlivera
test(servidor): usar number_format para convertir el precio a string
74810c2 
Al convertir un float a string, usando el cast (string) se eliminan los ceros de la parte decimal. Si se usa el método number_format se puede establecer los digitos de la parte decimal.

(string) 10.50 // "10.5"
number_format(10.50, 2, '.', '') // "10.50"
@MatiasOlivera
test(servidor): agregar la api de producto favorito
33bf896
@MatiasOlivera
feat(servidor): usar el gate administrar favoritos
3b79e8c
@MatiasOlivera
fix(servidor): corregir foráneas de la relación cotizaciones
3a53ece
@MatiasOlivera
feat(servidor): obtener las cotizaciones de un cliente
778fff2
@MatiasOlivera
feat(servidor): agregar la política de Cotizacion
d764d38
@MatiasOlivera
feat(servidor): agregar la política de CotizacionProducto
29e2eca
@MatiasOlivera MatiasOlivera added nueva característica New feature or request refactoring Further information is requested labels Feb 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@RomeoCirel RomeoCirel Awaiting requested review from RomeoCirel

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
nueva característica New feature or request refactoring Further information is requested
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Problemas en permisos y roles Autorización
1 participant
@MatiasOlivera