[DSI-7.5] Implementación de las Orders && [DSI-7.7] Arreglar el nuevo Admin Service para que trabaje con los microservicios && [DSI-7.8] Arreglar borrado de cuenta al borrar empresa - #64
Open
davposmar wants to merge 34 commits into
Conversation
- DemoSeed populates data-service db - DTOs - Company Settings in data-service - tables has been created in db - frontend uses data-service ( except claimRegister & update company & B2B )
- add endpoint to ask company name - add endpoint to ask orders by email
- frontend ordertype selector to avoid eager loading over all types - endpoint to get a map with key: orgId and value: orgName
- deferred load to get external company name
- data-service saves priority and locked
- this feature isn't really active
- Activity view is now using data-service - Admin administration section show correct information
- create a company must create company settings in data-service
- clear console.log - reaload user companies
- delete a company also deletes workers account when it's the last user worker entry
- when you deletes a worker, data-service delete the relationship between worker and unit to avoid orphan data
davposmar
added
documentation
…feat/data-service-orders
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Resumen
En esta pull request se separa la parte de orders del delivera-service y pasa a ser parte del data-service, con ello todos los datos relacionados con el FMS y por tanto la petición de ejecutar el FMS sobre los datos de nuestra compañía se hace sobre el data-service.
Se ha dockerizado tanto el data-service como el delivera-service con ello ahora se puede ejecutar directamente el docker-compose y tener disponible la aplicación, el único inconveniente es que tarda mucho dado que tiene que descargar e instalar el starter y tiene que descargar las dependencias necesarias para cada uno, en un futuro sería interesante reducir la cantidad de dependencias para que tarde menos y intentar reutilizar de alguna manera el starter para que no tarde tanto en construir las imágenes, ahora mismo con el docker-compose el escalado es imposible dado que se tienen que exponer puertos y están los nombres personalizados de los contenedores, sería interesante pasar a kubernetes en un futuro para poder escalar de manera automática y tener un balanceador de carga.
Se ha modificado el frontend para que trabaje de forma correcta con todos los microservicios pidiendo los datos necesarios a cada uno.
NO SE HAN ELIMINADO AÚN CONTROLLERS, SERVICES … DEL DELIVERA-SERVICE SE HARÁ CUANDO SE VAYAN A MIGRAR Y ACTUALIZAR LOS TEST.
Issues relacionados
#38 #41 #32
Cambios realizados
Tipo de cambio
Checklist