Mensajes
Añadir canvas
Archivos
CanvasListaCarpeta
Cargando historial...
Israel Naranjo
12:23
pos si te parece le das un aprobado ....
:marca_de_verificación_blanca:
1
Israel Naranjo
12:47
https://github.com/Tvlport-SPC/pos-web-fare-shop/pull/920 lo podrias revisar ?
Israel Naranjo
17:54
la aceptacion de https://github.com/Tvlport-SPC/pos-web-fare-shop/pull/920 lo podrias hacdr ?
Cristian Lorenzo
17:58
Sí
Israel Naranjo
18:00
:+1::tono-de-piel-5:
Cristian Lorenzo
18:03
La maquetación no está bien, dale un ojo
18:03
Sale todo super gigante
Israel Naranjo
18:03
no jodas si solo cambie un texto
Cristian Lorenzo
18:04
maq.png
maq.png
Israel Naranjo
18:04
otia
Israel Naranjo
18:35
el lunes se cerrara.... ya lo cambie
image.png
image.png
Cristian Lorenzo
18:35
Vale guay
Cristian Lorenzo
11:26
Si empizas https://travelport.atlassian.net/browse/EV-40483, no la muevas al sprint, que se ponen intensos con lo del planning
EV-40483 Define object graph and types
Status: To Do
Type: Story
Assignee: isnave
Priority: 3 - Medium
More actions...
:fuente_de_información: New: Click Notify to receive real-time issue updates directly in this channel.
Añadido por Jira Cloud
Israel Naranjo
11:34
hostia
11:34
lo dejo en el backlog
11:35
?
Cristian Lorenzo
11:35
Sí, ya lo moví yo. Es que creo que todo eso del planning y tal, es top-bottom
11:35
Viene de arriba y nos meterán un toque si nos desviamos
11:35
Si se hace d'estrangis, pues guay, pero no moviendo cosas si tenemos otras abiertas
Israel Naranjo
11:36
me parece bien
Cristian Lorenzo
11:36
Los extras, pues como hacíamos antes, lo movemos ya si conseguimos que esté en acceptance o hecho
Israel Naranjo
11:02
si tienes un rato para explicarme un poco por donde va la tarea
Cristian Lorenzo
11:02
Hay que sacar primero la de Manuel y la de Joan
11:02
Si esas estan, podemos empezar otras
Israel Naranjo
11:03
dime porque no he estado
11:03
me pongo con la que sea
11:03
joan estan 3 personas
Cristian Lorenzo
11:03
https://travelport.atlassian.net/browse/EV-40602
EV-40602 Display commitWorkbench error message correctly in the banner
Status: In Development
Type: Story
Assignee: Joan Perez camprubi
Priority: 3 - Medium
More actions...
:fuente_de_información: New: Click Notify to receive real-time issue updates directly in this channel.
Añadido por Jira Cloud
11:03
Y
11:03
https://travelport.atlassian.net/browse/EV-40589
EV-40589 Update data schema to use agency information APIs
Status: In Development
Type: Story
Assignee: Manuel Moraga
Priority: 3 - Medium
More actions...
:fuente_de_información: New: Click Notify to receive real-time issue updates directly in this channel.
Añadido por Jira Cloud
Israel Naranjo
11:04
me pongo con la de manu , con joan estan david y chema
Cristian Lorenzo
11:04
Vale, aquí puse lo que me pasó Núria https://teamtravelport.slack.com/archives/C05HX98J2V9/p1730704319351949
Cristian Lorenzo
Gracias, pero no es nada serio. Es un control normal de mútua + el de clínica
Publicado en team-rumiko | 4 nov. 2024 | Ver el mensaje
Israel Naranjo
11:08
la de manu pull request no veo
Cristian Lorenzo
11:08
No habrá empezado todavía
Israel Naranjo
11:08
oki
Cristian Lorenzo
10:47
Métele un screen shot como prueba
10:47
En un comentario
10:47
de lo del profile
Israel Naranjo
10:50
si si esta hecho y lo estoy comentando a melanie
Cristian Lorenzo
10:50
Súper, gracias
:+1::tono-de-piel-5:
1
Cristian Lorenzo
18:01
https://travelport.splunkcloud.com/en-US/app/search/search?q=search%20%22index%22%20%3D%20[…]rt&dispatch.sample_ratio=1&earliest=-30m%40m&latest=now
:+1::tono-de-piel-5:
1
Israel Naranjo
12:47
Cristian si tienes un rato, no encuentro el prroblema
Cristian Lorenzo
12:47
De momento complicado
Israel Naranjo
12:47
vale no problem
Israel Naranjo
12:00
te vienes a comer con nosotros o os invitan de nuevo
Israel Naranjo
12:15
estas disponible
12:15
??
Cristian Lorenzo
12:20
Si, a las 13
12:20
Tengo reunión ahora de media hora
12:21
Perdón 13:15
12:21
Son 45 min de reunión
Israel Naranjo
12:22
oki para aclararnos lo de las tareas relacionadas con el diseño tecnico
Cristian Lorenzo
12:24
Con el del state changes?
Israel Naranjo
12:26
con el [DRAFT] Save and restore Review and Confirm form state , para saber donde y como empezar las tareas que comenzaran Adrian y Chema que no esta claro
12:26
par no liarse
Cristian Lorenzo
12:26
Vale. A que hora coméis?
12:27
Yo luego de esto, me lo cuadro con vosotros que no tengo nada hasta las 16:00
Israel Naranjo
12:27
te parece que sobre las 14 14:30
12:27
?
Cristian Lorenzo
12:28
Vale
Israel Naranjo
12:28
mil gracias ,
Cristian Lorenzo
15:25
Hola, si pones captura y la versión del paquete en https://travelport.atlassian.net/browse/EV-39899, me avisas y lo acepto yo mismo
EV-39899 Invalid CustomerLoyalty SSRs are not rendered correctly in interact…
Status: In Acceptance
Type: Defect
Assignee: isnave
Priority: 3 - Medium
More actions...
:fuente_de_información: New: Click Notify to receive real-time issue updates directly in this channel.
Añadido por Jira Cloud
Israel Naranjo
15:40
oki voy
15:44
puesto
Israel Naranjo
14:47
ya que respondes este technical design https://travelport.atlassian.net/browse/EV-41459 es el de la pantalla itermedia que estuvimos hablando ??
EV-41459 Prepopulate middle screen for ancillaries with review and confirm d…
Status: In Development
Type: Story
Assignee: isnave
Priority: 3 - Medium
More actions...
:fuente_de_información: New: Click Notify to receive real-time issue updates directly in this channel.
Añadido por Jira Cloud
Cristian Lorenzo
14:47
Imagino que si
Israel Naranjo
14:49
no pone na y inacio esta chungo
Cristian Lorenzo
14:50
Tiene pinta de ser esa de los seats
Israel Naranjo
14:50
Gracias, a investigar
:+1::tono-de-piel-2:
1
Israel Naranjo
14:54
https://travelport.atlassian.net/wiki/spaces/EV/pages/326010387/Technical+Design+-+Persist+PAX+Info+in+Traveler+Info+Screen+to+the+PNR
en el punto 2 de los componentes me pones el comentario que hay una utilidad que se exporta para crear el showdialog ?
Technical Design - Persist PAX Info in Traveler Info Screen to the PNR
Context
This document details the technical design for a confirmation dialog that alerts users with a warning when they attempt to navigate to the seat selection screen from the...
Owned by: isnave | 5 comments | 7 people viewed
More actions...
Añadido por Confluence Cloud
Cristian Lorenzo
15:04
Sí, pero como decía, no creo que haga falta. Luego miramos
Cristian Lorenzo
16:16
Va bien adelantarlo a y 30?
Israel Naranjo
16:17
si
16:17
es algo rapido
Cristian Lorenzo
17:34
Plan Refactor de makeSeatMapsTravelerInformation.jsx para no usar el legacy plugin-traveler-information: (editado)
17:36
Ahora, Usa el selector getTravelerInformation de LPTI. Este ya no contiene los datos del formulario.
Ahora, Para hacer el commit, tambien usa una acción y un método de LPTI
17:36
Queremos:
17:38
Cuando esté EV-40646, utilice los datos que tenemos guardados del valor del formulario, el snapshot/transacción,
Si viene de Review, entonces hace mixin del formulario + PNR para poner unos default values.
En el submit, se usa setReservationTravelers de plugin-reservation-details-api-connection. Hay que verificar que las respuestas tienen la forma deseada.
Se elimina la dependencia con LPTI (no método de submit, no dispatch de acción).
(editado)
Israel Naranjo
09:32
me podrias validar el diseño ??? https://travelport.atlassian.net/wiki/spaces/EV/pages/326010387/Technical+Design+-+Persist+PAX+Info+in+Traveler+Info+Screen+to+the+PNR
Technical Design - Persist PAX Info in Traveler Info Screen to the PNR
Context
This document details the technical layout for navigating from the “Review and Confirm” screen to the “seat selection screen. Depending on whether the passenger informat...
Owned by: isnave | 6 comments | 7 people viewed
More actions...
Añadido por Confluence Cloud
Cristian Lorenzo
09:35
Voy
Cristian Lorenzo
09:44
Ya está. No nos centremos en los checks, que además son más sencillos: solo mira- y es seat maps quien lo hace como conocedor del dominio- si los PTCs de la oferta y las person casan
09:45
No miramos completeness porque no ganamos nada y solo movemos el check de sitio
Israel Naranjo
09:45
oki
Israel Naranjo
15:38
tienes un rato despues de la reunion?
Cristian Lorenzo
15:38
Sí, ahora a las 16:00
:+1::tono-de-piel-5:
1
Cristian Lorenzo
17:25
https://travelportprod.service-now.com/mytravelport_support?id=scorm_course&table=u_learning_course&search_course=07b4c8761bb7c5143c526535604bcb1f
Israel Naranjo
12:41
al final elpete del final me pasaba en 2 navegadores diferentes, en uno de ellos he borrado todo, y el pete nno me ha vuelto a dar en el que no he limpiado sigue dando .. curioso
Cristian Lorenzo
13:04
No se que esté full de traducciones
Israel Naranjo
13:10
donde puedo localizar el recurso DATE SEQUENCE IN ITINERARY NEEDS VERIFICATION, no lo veo en el github , lo has visto por casualidad ?
Cristian Lorenzo
13:10
Eso es de host
13:11
Supongo que tal cual
Israel Naranjo
16:37
lo puedes revisar
16:37
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/566
Israel Naranjo
18:05
https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/574
Israel Naranjo
10:18
si logras tener un rato que te rule y demas avisame please
Cristian Lorenzo
10:20
Si, estoy en ello con Dani
10:20
Me ha pasado el Zip por slack
Israel Naranjo
10:22
guay
Cristian Lorenzo
10:31
Me conecta pero no login luego con AD
Israel Naranjo
10:39
no te conecta
10:39
los certificados los has metido de nuevo
10:40
?
Cristian Lorenzo
10:40
Si conecta, pero era el login en AD para poder acceder a github, por ejemplo
Israel Naranjo
10:41
puffff
Cristian Lorenzo
11:26
Vale, parece que ahora sí. Y además pude salir de una reunión
Israel Naranjo
11:29
te llamo ?
Cristian Lorenzo
11:29
Vale
11:29
Con webex por si he de compartir yo algo
Cristian Lorenzo
11:56
Response fail commit:
{
"results": [
{
"error": {
"errors": [
{
"code": 12101,
"source": {
"pointer": "$travelerUpdates[0].travelerUpdatedItemQueryInput.TravelerUpdatedItem[0].PersonNameUpdatable"
},
"description": "Name change not allowed"
}
],
"name": "TravelportError",
"type": "RESPONSE",
"status": 424,
"message": "Error found in host response : NAME CHANGE NOT ALLOWED - CARRIER POLICY - CONTACT CARRIER"
},
"travelerId": "193ec09b-6321-460f-ab0e-ce5e41474588",
"success": false
}
],
"hasBeenCommited": false,
"commitErrors": null
}
11:57
Pero, no le llega al componente de error:
{
"workbench": {
"workbenchIdentifier": "d9b66f48-8f66-408c-80e6-c48c2b471d0e",
"reservationIdentifier": "d9b66f48-8f66-408c-80e6-c48c2b471d0e",
"offer": "[{…}]",
"travelAgency": null,
"traveler": "[{…}]",
"formOfPayment": "[]",
"travelerProduct": "[]",
"specialService": "[]",
"payment": "[]",
"documentOverrides": "[]",
"receipt": "[{…}]"
},
"mergedErrors": {
"results": "[]",
"hasBeenCommited": false,
"commitErrors": null
},
"lastError": null,
"travelerUpdatableItems": [
{
"__typename": "TravelerUpdatableItems",
"travelerIdentifier": "ae8887ea-e27e-4bad-974c-167de9a78c2e",
"id": "travelerUpdatableItems1",
"Identifier": {
"authority": "Travelport",
"value": "193ec09b-6321-460f-ab0e-ce5e41474588"
},
"TravelerUpdatableItem": [
"{CustomerLoyalty: null, Identifier: \"14c605eb-e524-…}",
"{Identifier: \"6a990c00-c1bd-4b85-a53c-0372706da034\"…}",
"{Identifier: \"49db85aa-0437-4779-b99d-d117e7b16039\"…}",
"{Comments: null, Identifier: \"3800a911-345d-425c-86…}",
"{Identifier: \"readonly\", PersonNameUpdatable: {…}, …}"
]
}
]
}
Cristian Lorenzo
12:03
Y para cliente, los props son,
{
"workbench": {
"workbenchIdentifier": "d9b66f48-8f66-408c-80e6-c48c2b471d0e",
"reservationIdentifier": "d9b66f48-8f66-408c-80e6-c48c2b471d0e",
"offer": "[{…}]",
"travelAgency": null,
"traveler": "[{…}]",
"formOfPayment": "[]",
"travelerProduct": "[]",
"specialService": "[]",
"payment": "[]",
"documentOverrides": "[]",
"receipt": "[{…}]"
},
"mergedErrors": {
"results": [
{
"error": {
"id": "",
"source": "",
"message": "Validation form failed. Some required fields are invalid or empty",
"type": "REQUEST",
"status": 400,
"data": [],
"name": "TravelportRequestExtendedError",
"setPropsForCSError": "setPropsForCSError() {}",
"setWorkflowId": "setWorkflowId() {}",
"errors": [
"{code: 26668, message: \"travelers[\"ae8887ea-e27e-4b…}"
]
},
"travelerId": "ae8887ea-e27e-4bad-974c-167de9a78c2e",
"success": false
}
],
"hasBeenCommited": false,
"commitErrors": null
},
"lastError": null,
"travelerUpdatableItems": [
"{Identifier: {…}, TravelerUpdatableItem: Array(5), …}"
]
}
12:04
Mirar por qué los filtra, y si los deja entrar, por que no se renderiza bien
12:04
plugins/plugin-traveler-information-gui/src/edit-travelers-feature/components/EditTravelerErrorOutlet/filterErrorsNotToBeDisplayed.ts
Israel Naranjo
13:36
puedes echarle un ojo
Cristian Lorenzo
13:37
El que? Me pasas un enlace?
Israel Naranjo
13:38
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/571
13:38
rozando el limnite
Cristian Lorenzo
13:39
Por mi guay si funciona
13:39
Luego ya miramos de simplificar, como dijimos en la DSU
Israel Naranjo
13:39
si si
Cristian Lorenzo
13:39
Pues adelante
13:39
Dadle un test a mano grande, con R&C de varios tipos, fallos de payment en los R&C, y el edit
Israel Naranjo
13:40
lo probe con el tunnel pero en cuanto este subido lo pruebo de nuevo
Cristian Lorenzo
13:40
Súper, gracias
13:40
Salgo un momento que han de cortar las uñas a las perras antes de que salga de viaje
Israel Naranjo
13:40
oki
13:40
merci
Israel Naranjo
11:46
donde dices que esta el metodo q faya ??
Cristian Lorenzo
11:47
packages/non-air-modify-reservation-utils/src/traveler-editable-object-mappings/makeUpdateInputFromTravelerEditableObject/getUpdateObjectIfDirty.ts
11:47
En general, todo lo de packages/non-air-modify-reservation-utils/src/traveler-editable-object-mappings/makeUpdateInputFromTravelerEditableObject
Israel Naranjo
11:48
pero donde no se filtra es en el submit del formulario que manda todo , no crees ??
Cristian Lorenzo
11:48
No, ha de ser ahí
11:48
El formulario le manda su estado
11:49
Pero preparar las llamadas a los servicios no es su responsabilidad
Israel Naranjo
11:49
vale no lo entendia asi , voy a ojear
Israel Naranjo
15:14
cristian la historia esta que en filterNullishValues.ts nos cepillamos las propiedades con null non-air-modify-reservation-utils getUpdateObjectIfDirty sourceEmail[field] !== pristineEmail[field] en pristineEmail contiene las propiedades null por lo que falla las comparaciones
Cristian Lorenzo
15:17
Igual hay que mirar que pase con nullish y no solo null
Israel Naranjo
15:19
entonces hay que pasar de filterNullishValues
15:19
y dejar pasar los que dices
Cristian Lorenzo
15:20
Pero puede que sea solo en email y phone
Israel Naranjo
15:20
de momento estos 2
Cristian Lorenzo
15:21
Usando lo de checkEqualWithNullishCast
15:21
En este archivo packages/non-air-modify-reservation-utils/src/traveler-editable-object-mappings/makeUpdateInputFromTravelerEditableObject/getUpdateObjectIfDirty.ts
15:21
Para TravelerUpdatableItemEmail y TravelerUpdatableItemTelephone
Israel Naranjo
15:22
pos-web-traveler-details ya lo manda mal
Cristian Lorenzo
15:23
Yo no tocaría nada de momento en traveler-details
15:23
Miremos si con lo que comento en reservation details ya va
Israel Naranjo
15:23
oki haber
Israel Naranjo
10:49
Cristian dale un ojo https://travelport.atlassian.net/browse/EV-43494
EV-43494 To many e-mails and phones are sent after updating booking data
Status: In Acceptance
Type: Defect
Assignee: isnave
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud
:marca_de_verificación_blanca:
1
Israel Naranjo
16:52
Cristian , este repo creo que deberia actualizar las dependencias, al menos la de telemtry y logging
fes-profiles-manager
"@pos-web/profiles-manager-data-types": "1.4.3",
"@pos-web/profiles-manager-entities": "1.8.3",
"@pos-web/travelport-logging-node": "2.1.1",
"@pos-web/travelport-telemetry-node": "2.3.0"
Cristian Lorenzo
16:53
No las sube el renovate?
Israel Naranjo
16:53
esta degradado
16:53
y viendo con manrrique no esta la que el hizo el otro dia
Cristian Lorenzo
16:58
Vale. Será el import, se cargaron el paquete
Israel Naranjo
17:02
se lio parda
17:07
https://github.com/Tvlport-SPC/fes-profiles-manager/pull/77 echale un ojo please
Cristian Lorenzo
17:29
Si la nueva imagen no levanta, igual hay que cambiar como se hace el import
17:29
Espero que no
Israel Naranjo
17:29
espero q no ufff
17:31
ese mismo repo esta igual en dev1
Cristian Lorenzo
17:31
No lo puede levantar
17:34
En otros repos es "@pos-web/travelport-telemetry-node": "1.16.6",
17:34
Y lo que pasa es que el import default funciona distinto
Israel Naranjo
17:36
si es el lio de la semana pasada
17:36
que cambiaron los require por import
Cristian Lorenzo
17:37
Voy a ver
Cristian Lorenzo
17:53
Si pasa las pipelines, igual lo soluciona https://github.com/Tvlport-SPC/fes-profiles-manager/pull/78
#78 fix: import using default
Description
Is this change related to a Jira Task?
• Yes
• Story Link:
• Feature Link:
• No
PR Checklist
Please check if your PR fulfills the following requirements:
• The commit message follows our guidelines.
Mostrar más
Tvlport-SPC/fes-profiles-manager | 20 feb. | Añadido por GitHub
Israel Naranjo
18:04
el sonar
18:06
na lo lance de nuevo
18:10
hago el merge??
Cristian Lorenzo
18:11
Sí, adelante
Israel Naranjo
18:24
na
node -r @pos-web/travelport-telemetry-node/dist/travelport-telemetry-node.cjs.js ./dist/src/index.js
node:internal/modules/cjs/loader:1143
throw err;
^
Error: Cannot find module '@pos-web/travelport-telemetry-node/dist/travelport-telemetry-node.cjs.js'
Require stack:
internal/preload
at Module._resolveFilename (node:internal/modules/cjs/loader:1140:15)
at Module._load (node:internal/modules/cjs/loader:981:27)
at internalRequire (node:internal/modules/cjs/loader:174:19)
at Module._preloadModules (node:internal/modules/cjs/loader:1569:5)
at loadPreloadModules (node:internal/process/pre_execution:628:5)
at setupUserModules (node:internal/process/pre_execution:154:3)
at prepareExecution (node:internal/process/pre_execution:114:5)
at prepareMainThreadExecution (node:internal/process/pre_execution:40:10)
at node:internal/main/run_main_module:11:19 {
code: 'MODULE_NOT_FOUND',
requireStack: [ 'internal/preload' ]
}
Node.js v18.20.6
Cristian Lorenzo
18:25
Esa no es la imagen, no?
Israel Naranjo
18:25
si es
18:26
:sudor:
18:27
leñe se corrigio
18:27
haber si carga rumiko
Israel Naranjo
18:33
na error
Cristian Lorenzo
18:33
Si carga bien. Pero tiene ooo
Israel Naranjo
18:33
Sorry, there was an issue connecting you
18:33
no te sale eso ?
Cristian Lorenzo
18:38
No. Ojo con la pipeline de CD que igual te tira atrás los deploys
18:40
Cuando hace el rollback
18:40
Por eso yo la suelo cancelar en estos que siempre falla
Israel Naranjo
18:40
hostia pero no se ve fallo
18:40
puffff
Cristian Lorenzo
18:48
No hace falta lo tires, es que tarda un huevo en poner el que tiré yo hace 10 mins
Israel Naranjo
18:48
jurrrr
18:48
y llevas razon casco el mockoon
Israel Naranjo
11:35
Cristian disculpa si te he llegado a molesta en la daily, no era mi intención.:sudor:
Cristian Lorenzo
11:36
No, tranquilo.
11:36
Lo mismo digo
Israel Naranjo
11:39
Para nada
Israel Naranjo
16:55
No entiend a la gente
Cristian Lorenzo
16:55
Si me toca, pago, si no, no
16:56
Pues yo no pago si no es desde el principio
Israel Naranjo
16:56
me pasa lo mismo , pero es qu es lo que va a pasar
16:56
que la gente quieren que pagen los que les tocan
16:56
y fiate de que paguen los 100
Cristian Lorenzo
16:57
Y la mentalidad tacaña. Mi mujer a los clientes así los echa del salon
16:57
Yo me pensaba que iba a ser mucho mucho más
Israel Naranjo
16:58
por 100 euros no es dinero , como se puede ser tan chungo
16:58
Nuria no no pero si y el tal Soul igual
16:59
Perdon Laura
Israel Naranjo
14:33
has hecho lo de los abogados ???
Cristian Lorenzo
14:33
No, todavía no. A la noche en casa
14:34
Pero poco optimista soy
14:34
De que lleguemos
Israel Naranjo
14:35
ya el pobre Victor este llevando el tema de los abogados , habir cuantos somos y si lo puede negocias
Israel Naranjo
16:00
Cristian pillamos el owner de pos-web-plugin-sell ?? estamos todo el rato modificandolo y tenemos que andar con arya para que nos lo aprueben
16:01
he hablado con Oscar Salcedo para ve que les parece
Israel Naranjo
16:22
:hola:
Cristian Lorenzo
16:26
Vale, me parece bien
Israel Naranjo
13:07
cuando puedas mirar la pr haber si la saco
Cristian Lorenzo
13:08
Estoy en una llamada
Israel Naranjo
13:08
oki sorry
Israel Naranjo
15:36
cristian lo del david
15:36
esta relacionado con lo que has subido ?
Cristian Lorenzo
15:36
Sí. Estoy creando el dataset
15:37
Y luego ya se puede cambiar sobre lo que hizo David
Israel Naranjo
16:10
Cristian en la pull de David veo que esta la tarjeta la featureflag , no veo q faltaria por hacer?
Cristian Lorenzo
16:55
Ya no se va a meter feature flag
16:55
Ira por un reference data
16:56
Si quieres te explico, o si no mañana si vas a la oficina
Israel Naranjo
16:57
mejor en persona mañana
Cristian Lorenzo
16:57
Vale
16:57
A ver si me da tiempo a tener el reference data listo, que es un rollazo inmenso
Israel Naranjo
16:58
si se te puede ayudar tu diras
Cristian Lorenzo
16:58
Buenos, los PRs esos. Son cositas sencillas
16:58
Pero como toca 4 componentes, pues se alarga
Cristian Lorenzo
17:06
Vale, pues ya está disponible
17:06
Así mañana nos podemos poner directos a ello
Israel Naranjo
17:11
Genial
Israel Naranjo
17:53
Cristian me puedes echar un ojo a la pull , no consigo que me tire los test https://github.com/Tvlport-SPC/pos-web-payment/pull/414
Cristian Lorenzo
17:56
El yarn lo tienes desactualizado
Israel Naranjo
18:00
leñe lo he pasado tela de veces
18:03
todo el rato -> algo se me pasa {
"status": "EMPTY",
"error": null,
"data": null
}
Cristian Lorenzo
18:04
Haz prefech en el beforeEach llamando el método o haz un waitFor
18:04
Es asíncrono y antes no lo era
Cristian Lorenzo
18:10
Luego hay que añadir tambien el plugin-reference-data cuando se añade el plugin
18:10
Y ya a partir de ahí tira pero hay errores de programación
Israel Naranjo
18:14
el plugin-reference-data te refieres a testWithPluginReferenceData y PluginReferenceDataPkg, PluginReferenceData
Cristian Lorenzo
18:15
Si, depende de cómo lo cargue. En un test lo hace a la vieja usanza de addPlugin
18:15
Y en ese addPlugin pues falta el reference data
18:16
En los sitios que se use testWith, y para la test util publica, pues también hay que añadirlo
Israel Naranjo
18:17
testWith le tengo puesto testWithPluginReferenceData como otros sitios q vi pero ne el addplugin no , y hay que meterlo ??
18:17
le tengo puesto addPlugin(PluginReferenceDataPkg, PluginReferenceData);
Cristian Lorenzo
18:18
Si, porque si no te dirá que la Orión Key es de un plugin inactivo.
Israel Naranjo
18:18
oki
Israel Naranjo
11:36
Cristian
Cristian Lorenzo
11:37
Hola
Israel Naranjo
11:37
la tarea del fes-traveler-information, refactorizar que implce . para no liarme ha hacer el loco
Cristian Lorenzo
11:37
Hacer un nuevo servicio con el diseño técnico que preparamos en su momento.
11:38
Ponemos un meeting a las 12:30?
Israel Naranjo
11:38
mejor
11:38
la tarea tiene 0 info y prefiero ir sobre seguro
Cristian Lorenzo
11:38
Vale
Israel Naranjo
12:31
estas??
Cristian Lorenzo
12:31
Sí
12:31
Llamada Webex?
Israel Naranjo
12:31
sipi
Israel Naranjo
12:49
leñe el bug que comentan de las tarjetas?
12:50
quieres que lo mirro yo ?
Cristian Lorenzo
13:00
Dice Adri que lo mira
:+1:
1
Israel Naranjo
12:18
Cristian cuando puedas echar un ojo https://github.com/Tvlport-SPC/fes-traveler-information-services
Cristian Lorenzo
12:19
Hay demasiada cosa, los controladores no los sabemos. De momento se empezará con la ruta de upsert del diseño, el resto está por ver.
12:21
Empecemos por repo vacío, crear una carpeta de feature de traveler-reconciliation
12:21
Ahí dentro los interfaces, el README.md, y los feature files
12:22
Y con esos feature files se hace el testingdriver
Israel Naranjo
12:22
el repo como le llamo
12:23
traveler-reconciliation??
Cristian Lorenzo
12:23
No, porque haremos tambien lo de agencia
Israel Naranjo
12:23
fes-traveler-information-services este te vale ?
Cristian Lorenzo
12:24
Tampoco me va, porque es más que traveler information. No sé la verdad
12:25
Igual fes-reservation-travelers-agency-manager, pero no estoy seguro. Lo podemos por eso decidir más adelante
Israel Naranjo
12:26
oki una duda se trabaja sobre main? o se hace un primera subida y un branch para modificar
Cristian Lorenzo
12:26
Con branches mejor
Israel Naranjo
12:26
vale
Israel Naranjo
09:53
https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/1
09:53
le podrias hojear
Cristian Lorenzo
09:54
Esta guay, pero concentémosnos en src/services/TestReservationMutationService.ts
09:54
Este ha de implementar el interfaz
09:54
Y ha de tener un feature file asociado
09:55
Los otros sobran
09:55
Los modelos, si son los mismos que implementados en zod
09:55
No hace falta declararlos de nuevo. Zod tiene inferencia de Typescript
09:56
Con lo que suele ser algo type Reservation = Infer, la fórmula exacta depende de la biblioteca de validación en sí
09:56
Déjame mirar
Israel Naranjo
09:56
si
09:56
el infer lo use
Cristian Lorenzo
09:57
Guay, pues eso. Me refiero a que con eso, archivos como src/models/Reservation.ts debieran ser redundantes
09:59
Luego la parte de los services, igual debiera estar en su propia carpeta de feature (editado)
09:59
Tipo traveler-management, o en la de reconciliation
09:59
Y agency pues se puede poner también así en su carpeta
Israel Naranjo
11:44
puedes echarle un ojo para ver si estoy muy desencaminado
11:44
???
Cristian Lorenzo
11:46
Para que usamos el override?
11:47
Y la especificacion es muy particular del driver
Israel Naranjo
11:54
vale lo puse para si alguno lo necesita sobreescribir el codigo
11:54
y no repetir codigo
11:55
pero si ves quepuede ser un foco de problema se pone normal
Cristian Lorenzo
11:58
Es que estamos adelantando problemas que igual no existen
11:58
O que requieren otra solución como sea factoría, o herencia de prototipo normal
11:59
Todo lo que no sea el spec, el interfaz, y el testing driver es demasiado
Israel Naranjo
11:59
:sudor: oki lo dejo sin cosas raras
12:02
me traigo el codigo de mappingHelper?
Cristian Lorenzo
12:03
No. Este servicio usa interfaces tipo JSON
Israel Naranjo
12:05
para comparar entre el AAA y el estado deseado pone de usar un metodo deribado de d mappinghelper (editado)
Cristian Lorenzo
12:05
el triple AAA se lee mediante workbench
12:06
Si usas esos interfaces añades una dependencia con fes-pnr-retrieve i cs-pnr-retrieve o como se llamen
12:06
Que son los que publican esos datos
Israel Naranjo
12:07
lo que pone en el spike
Cristian Lorenzo
12:07
Nosotros queremos un servicio que gestiones upserts, en particular que los haga usando JSON API sin necesidad de más interfaces que las publicadas ts-comms layer
12:07
Que pone en el spike?
Israel Naranjo
12:08
A method, partially derived from the mappingHelper of fes-traveler-information, will generate a list of operations required to transform the
initial state into the desired state:
Cristian Lorenzo
12:08
Sí, en la lógica de comparar, pero no en interfaces. Las operaciones son las mismas (añadir, editar, borrar).
Israel Naranjo
12:09
por eso te decia de usar ese codigo parr comparar
Cristian Lorenzo
12:10
No puedes usar ese código directamente para comparar
12:10
Porque los interfaces son distintos. Tienes que portar la lógica
Israel Naranjo
12:10
oki es lo que me referia,
Cristian Lorenzo
12:11
Porque el objeto a comparar es una reserva, es decir un workbench, y el input es el upsertTravelers o como se llame que estamos usando en reservation-details y las vistas de review and confirm y gbf
Israel Naranjo
16:41
puedes mirar la pr , https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/1
16:41
:dedos_cruzados:
Cristian Lorenzo
16:43
Sobran cosas
16:43
Primer PR sin nada de los reservation service que no sea el test
Israel Naranjo
16:44
el fichero de test TestReservationMutationService.ts es el que he implementado
Cristian Lorenzo
16:45
Está todo lleno de queries graphql y cosas por el estilo
Israel Naranjo
16:45
en ese fichero ???
Cristian Lorenzo
16:46
La PR tiene 73 archivos
16:46
src/services/CSReservationMutationService.ts (editado)
16:46
Por ejemplo
Israel Naranjo
16:46
eso va fuera ahora
Cristian Lorenzo
16:46
O src/graphql/dataSources.ts
Israel Naranjo
16:46
todo eso lo quito
Cristian Lorenzo
16:47
Cuando lo tengas lo miro
Israel Naranjo
16:47
:cara_con_ceja_levantada:
Israel Naranjo
11:46
Compañero, me revisas la pr porfa
Cristian Lorenzo
11:46
Es el 2 o el 1
Israel Naranjo
11:48
1
Cristian Lorenzo
12:00
Mueve las cosas a carpetas de features
12:01
No ha de haber nada en el ráiz que no sea setup/levantar el servicio
Israel Naranjo
12:28
alfo asi ??
image.png
image.png
Cristian Lorenzo
12:29
Sí. Rompe tambien lo de pillar/refrescar el workbench del resto del servicio
12:30
E igual lo de prefijar las feature folders, pero eso más opcional
Israel Naranjo
08:50
he subido todos los cambios , por donde tirarias ahora ?
Cristian Lorenzo
08:59
Empezar con el reconciliador, en particular la parte que calcula cambios hay que hacer a una reserva para obtener el estado deseado
09:01
Para las dependencias, usar un inyector o un patrón singleton
09:01
Así es más fácil de gestionar e inyectar mocks
Cristian Lorenzo
09:17
Luego revisamos un patron modular. Lo usé para la prueba y creo que va bien para simplificar setups
Israel Naranjo
09:17
genial
Israel Naranjo
09:24
seria algo como esto ??
src/
├── modules/
│ ├── traveler/
│ │ ├── _tests_/
│ │ ├── domain/
│ │ │ ├── entities/
│ │ │ └── interfaces/
│ │ ├── infrastructure/
│ │ │ └── repositories/
│ │ ├── application/
│ │ │ └── services/
│ │ └── index.ts
│ ├── workbench/
│ │ ├── _tests_/
│ │ ├── domain/
│ │ ├── infrastructure/
│ │ ├── application/
│ │ └── index.ts
├── core/
│ ├── config/
│ ├── server/
│ └── types/
└── shared/
├── utils/
└── interfaces/
Cristian Lorenzo
09:25
Bien. Que iria en infrastructure?
Israel Naranjo
09:26
import { ITravelerRepository } from '../../domain/interfaces/ITravelerRepository';
import { Traveler } from '../../domain/entities/Traveler';
export class TravelerRepository implements ITravelerRepository {
async findById(id: string): Promise {
// Implementation
}
async save(traveler: Traveler): Promise {
// Implementation
}
}
Cristian Lorenzo
09:28
Por desgracia no podrás aplicar ese patrón directamente
09:28
No puedes hacer Entity/Manager tal cual representando filas en base de datos
09:29
Y tendrás que inyectar el modulo de workbench
09:30
Y pasarle el workbenchid o abstraerlo bajo algun tipo de concepto genérico de transacción
09:30
Pero sí, el orden está bien
09:30
Los controllers también irian dentro del dominio
Israel Naranjo
09:32
no irian dentro de infraestructura??
Cristian Lorenzo
09:33
Vale, no sé que significa de antemano las carpetas. Si es consistente me parece bien
Israel Naranjo
09:33
hago esos cambios o me dedico a lo que comentaste al principio
Cristian Lorenzo
09:34
Haz merge y puedes hacer otro PR con esos cambios
09:35
Y luego ya te pones. Me parece bien la idea. Documenta en readme.md principal la estructura de carpetas para que haya consistencia a largo plazo y ya está
Israel Naranjo
09:35
muchas gracias
Israel Naranjo
15:52
cristian le puedes echar un ojo https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/4
Cristian Lorenzo
15:53
Está bien. Si vamos a usar node sin transpilar, por eso, necesitamos usar extensiones de archivos en los imports
15:53
Pero de momento vale
Israel Naranjo
15:59
se debe poner la extension del archivo ? .ts
15:59
para el import
Cristian Lorenzo
15:59
Sí. Ya verás, prueba a lanzar con node --experimental-transform-types
15:59
Y fallará
Israel Naranjo
16:23
oki vvoy a probar
16:24
hablando con la gente , te animas a firmar para representante??
16:24
y crear el proyecto en sonar
16:25
puedo ?
Cristian Lorenzo
16:27
Firmar para que se puedan hacer elecciones?
Israel Naranjo
16:27
par una ccandidatura
Cristian Lorenzo
16:27
El proyecto creo que se pide con service now
Israel Naranjo
16:28
oki cascaba antes
Cristian Lorenzo
16:29
Yo si hace falta para que alguien se presente o se hagan elecciones, vale. Presentarme no
Israel Naranjo
16:33
ñaccc, ya que lo he promovido me presentare........
Israel Naranjo
12:12
podrias mirar la pr https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/4 antesd e que te pires
12:12
:rezo:
Cristian Lorenzo
12:14
Yo te recomiendo a ver si es posible usar algo que no dependa del reflect-metadata
12:14
Porque vas a necesitar compilar typescript
12:14
Y si puedes usar node directamente, mejor
12:15
Y hacer PRs más pequeñas y frecuentes
Israel Naranjo
12:20
cierto.
Israel Naranjo
12:25
al cambiar eel result tuve que cambiar el copon de cosas por no mirar
Israel Naranjo
13:20
te parece que ponga la pull request para que la mira y se procede se meta en main
Cristian Lorenzo
13:21
Como veas. Recomiendo encarecidamente lo de no depender de reflect-metadata. Y usar en su lugar una biblioteca de DI más verbosa o un patrón singleton con constructores
Israel Naranjo
13:22
he quitado toda la dependencia
Cristian Lorenzo
13:22
Vale
Israel Naranjo
11:50
Cristian , crees que si me hago una pull con el esqueleto y poco a poco sera mejor , no ?
Cristian Lorenzo
11:51
Sí, siempre es mejor PRs más pequeñas
:+1:
1
Israel Naranjo
10:28
Cristian cuando tengas un rato le puedes echar un ojo a la pull, y me dices lo que te parece y de cambiar lo que sea .
Cristian Lorenzo
10:29
Que pull?
Israel Naranjo
10:29
https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/9
Cristian Lorenzo
10:32
Yo te recomiendo que no uses singleton en clases y emplees constructores con dependencia explícita
10:33
Si no vas a usar un framework de DI
10:33
Y que no mockees imports
Israel Naranjo
10:34
quitare los singleton ahora
Cristian Lorenzo
10:34
El singleton está bien, pero no en las clases.
10:35
Y el servicy registry no debe saber que tipos hay registrados, eso debe ser lazy
10:35
Creo que también te comenté que const TSV_VERSIONS = ['', '/TSV11']; no tiene sentido en servicios nuevos
Israel Naranjo
10:35
cierto
Cristian Lorenzo
10:35
Es algo que debe ser transparente para los consumidores
Israel Naranjo
10:36
lo dejare en blanco porque no se cual es exactamente
Cristian Lorenzo
10:36
No ha de ser ninguno
10:37
Para los consumidores de este servicio, como se consiga hacer una transacción es indiferente
10:37
Si va por TS, por Traveldata, terminal, uAPI... no han de tener conocimineto de ello
10:37
Y por tanto no han de pasar la versión de TS por la URL
Israel Naranjo
10:38
apunto
10:42
que andas haciendo jodio , en que andas liado (editado)
Cristian Lorenzo
10:42
Estoy mirando como reducir los tiempos de CI, pero me he estancado un poco en storefront
Israel Naranjo
10:47
asi estas entretenido
Cristian Lorenzo
10:47
Bueno, no es muy guay esto en concreto
10:47
Pero salen cosas que ayudarán a todo el mundo
Israel Naranjo
10:48
sobre todo busca cosas que te motiven
Cristian Lorenzo
10:53
Sí, a ver si hay cosas guays. Pero hace falta un cambio de arriba a bajo.
Israel Naranjo
11:02
si tio, no se haber si con tu ayuda pega esto un cambio . ojala tio
Cristian Lorenzo
11:23
No sé, es de más arriba que hace falta cambiar.
Israel Naranjo
11:31
pufffff , lo sigo viendo igual . Como pollo sin cabeza
Cristian Lorenzo
11:34
Son cosas de espíritu, de querer trabajar con calidad
Israel Naranjo
10:15
Hoy no curras ?
Cristian Lorenzo
10:15
No, hoy puente
Israel Naranjo
10:15
haces bien
Israel Naranjo
09:41
Buenos dias Cristian , tio me podrias revisar https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/12 haber si se cierra el tema
Cristian Lorenzo
09:41
Esto cada vez tiene más archivos
Israel Naranjo
09:43
ya , si me puedes echar un cable ,
Cristian Lorenzo
09:52
Creo que es excesivo el sistema del registro y hay un montón de features que igual se usan o no
09:52
También hay comentarios en castellano
Israel Naranjo
09:52
no jodas que se me han colado
Cristian Lorenzo
09:53
Y se mezclan cosas, no?
09:53
Inyección directa + registry
09:53
Por ejemplo src/modules/traveler/application/services/TestReservationMutationService.ts
09:56
El logger que habrás de integrar es el node-logger de travelport, no directamente un adaptador de winston
Israel Naranjo
09:56
El código busca dinámicamente el método de adición de viajeros en el repositorio para poder simularlo en las pruebas, par evitar errores por nombres de métodos incorrectos
09:57
dude , porque Irene casco ese
Cristian Lorenzo
09:57
Wiston es el más común, pero tendrás que integrar el de travelport
Israel Naranjo
09:58
si tienees un rato despues , para charlar
Cristian Lorenzo
09:58
Sí tengo
Israel Naranjo
09:58
que meten mano nadie lo quiere mirar
09:58
pues cuando tu me digas tengo la mañana libre
Cristian Lorenzo
09:59
A las 11:00 acaba la reunión de Burton con Mattieu
Israel Naranjo
10:00
pues dame un toque cuando termines
Cristian Lorenzo
11:11
Tengo una llamada con Kike y luego miramos
Israel Naranjo
11:11
perfecto tio , gracias
Cristian Lorenzo
12:02
Ya estoy (editado)
Israel Naranjo
12:03
te llam
Cristian Lorenzo
12:03
OK
Israel Naranjo
09:47
cristian de este commit 4b6d3cf2 se han borrado carpeta y archivos , hay algun motivo ??
Cristian Lorenzo
09:47
No sé de què me hablas
09:48
Commit de que repo?
Israel Naranjo
09:48
https://github.com/Tvlport-SPC/pos-web-traveler-details
09:48
sorry q te doy la info a medias
Cristian Lorenzo
09:50
Estoy llegando a la oficina. Puede ser todo lo que hay que quitar y quitado del antiguo review and confirm
Israel Naranjo
09:50
vale nada cunado puedas,
Cristian Lorenzo
09:50
De hecho todo un plugin se tiene que quitar si no lo hice ya
Israel Naranjo
09:51
esta a medias de borrar plugins\plugin-traveler-details
09:51
y no tira
Cristian Lorenzo
09:52
Ahora me explicas cuando llegue
Israel Naranjo
09:52
si si perfecto tio
09:52
no he ido a la ofivcina tengo que salir pitando a comprar una nevera que se mae ha jodido el congelador
Cristian Lorenzo
10:01
Vale, ya estoy ahí. Que problema hay?
Israel Naranjo
10:15
estoy tratando de hacer la https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/646
Cristian Lorenzo
10:15
Hay que instalar más paquetes
10:15
@pos-web/results-system
Israel Naranjo
10:15
y si te fijas la carpeta del plugun esta a medio eliominar
Cristian Lorenzo
10:16
Sí, no se puede eliminar completo porque hay que hacer una PR a hoteles
10:16
Pero lo quitado, bien quitado está
Israel Naranjo
10:16
sino no pasan ni los test porque busca el package.json
Cristian Lorenzo
10:17
Todos los test fallan por lo mismo
10:17
Hay que instalar el paquete que digo
10:17
Cannot find module '@pos-web/results-system' from 'node_modules/@pos-web/plugin-storefront-api-connection/dist/@pos-web/plugin-storefront-api-connection.umd.js'
3139
3140
Require stack:
3141
node_modules/@pos-web/plugin-storefront-api-connection/dist/@pos-web/plugin-storefront-api-connection.umd.js
3142
node_modules/@pos-web/plugin-storefront-api-connection/test-utils.js
3143
node_modules/@pos-web/plugin-fare-shop-search/test-utils.js
3144
node_modules/@pos-web/plugin-fare-shop-gui/test-utils.js
3145
node_modules/@pos-web/plugin-pnr-gui/test-utils.js
3146
node_modules/@pos-web/plugin-reservation-details-gui/test-utils.js
3147
plugins/plugin-hotel-traveler-details/src/test-utils/testWithPluginHotelTravelerDetails.ts
3148
plugins/plugin-hotel-traveler-details/src/test-utils/internalTestWithPluginHotelTravelerDetails.ts
3149
plugins/plugin-hotel-traveler-details/src/__tests__/PluginHotelTravelerDetails.spec.ts
10:17
No tiene nada que ver con el plugin en concreto
Israel Naranjo
10:17
hasta q no borre la carpeta ni se me ejecutaba
Cristian Lorenzo
10:18
Las carpetas vacias cuando se elimina un plugin son otra cosa. Pero la pipeline falla por lo que te digo
10:18
Siempre va a quedar un residuo con los archivos no seguidos como la carpeta dist
10:18
Pero no tiene que ver con la pipeline
Israel Naranjo
10:19
me rayaba que no se ejecutasen porque no encontraba el json
10:20
Tendra un rato para reunirnos , para hablar de las tareas pendientes de la refactorizacion que el Nacho me esta calentando ???
Cristian Lorenzo
10:20
Sí. Ponme cita cuando esté libre durante el día
10:20
Pilla una hora o dos y miramos
Israel Naranjo
10:21
ahora mismo tienes para q te meta en la dailly
10:21
???
Cristian Lorenzo
10:21
La daily de Burton es de 10:30 a 11
Israel Naranjo
13:13
a las 3 tienes hueco ???
Israel Naranjo
13:32
Compiiii
Cristian Lorenzo
13:42
Si arreglo eso que peta de PNR sí. Esto de los imports de Orion no lo pilla ni el tunel ni la CI y ahora se rompió
Israel Naranjo
13:44
su me dicesw otra hora ??++
Cristian Lorenzo
13:44
Creo que si estará
13:44
Pero a ver...
13:48
Le puedes dar a la PR https://github.com/Tvlport-SPC/pos-web-pnr/pull/1721?
#1721 fix: solve import errors caused by embedded helper
## Description
Solve import errors caused by embedded helper
## Is this change related to a Jira Task?
• Yes
• Story/task Link:
• Feature Link:
• No
## PR Checklist
Please check if your PR fulfills the following requirements:
• The commit message follows our guidelines.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-pnr | 6 may. | Añadido por GitHub
Israel Naranjo
13:49
done
13:49
te peta el sonnar
Cristian Lorenzo
13:49
Gracias. Sí ,es que no tiene coverage ese repo
13:49
Ya hicieron merge en su día
13:50
No voy a arreglar eso ahora
Israel Naranjo
13:50
:sonrisa_con_sudor:
Cristian Lorenzo
14:06
Vale, ya va
14:06
confirmo la reunión
Israel Naranjo
14:14
Thank
Israel Naranjo
16:40
Gracias por la ayuda Cristian
Cristian Lorenzo
16:46
De nada
Israel Naranjo
13:03
donde vas a comer?
Israel Naranjo
11:21
no has estado en la reunion para lo del comite (editado)
Cristian Lorenzo
11:22
Tenía que estar disponible para el perito del seguro de la finca
Israel Naranjo
11:22
han comentado que van a cerrar la oficina de madrid
11:22
no se si en un mes o asi
Cristian Lorenzo
11:23
Que los ponen en remoto o ya no queda nadie allí?
Israel Naranjo
11:23
nno han dicho nada en concreto
11:23
pero huele a mierda por donde mires
Cristian Lorenzo
11:23
Ya
11:24
El plan industrial de la oficina de Barcelona es una kk
Israel Naranjo
11:24
el contratode la ofi es hastael 2026 haber que pasa
Cristian Lorenzo
11:26
Bueno, no hablo del edificio, si no del proyecto. Sin proyecto el edificio es nada, pueden incluso pagar y salirse antes si quieren
11:27
A ver si me dan ya fecha de inicio que eso me tiene mosca
Israel Naranjo
11:28
cierto , y lo de Microsoft es un putadon estar esperando
Cristian Lorenzo
11:28
Sí, la verdad. Espero que sea que el tipo estuvo dos semanas de vacaciones
11:28
Y no se demore. El martes les vuelvo a insistir a ver
Israel Naranjo
10:03
curras hoy
10:03
??
Cristian Lorenzo
10:32
No de momento, tengo una urgencia veterinaria
Israel Naranjo
09:09
qtal fue todo
Cristian Lorenzo
09:11
No pinta bien, la verdad. Esta noche volvió a vomitar, le han puesto un medicamento adicional y parece que lo controla, y toca hacer análisis de control hoy. Pero parece que la vesícula a sangrado por algun motivo y eso está trastocando todo.
09:11
Sumale lo que tiene por culpa de la leishmania, que puede o no estar relacionado
Israel Naranjo
09:11
pobrecita
Cristian Lorenzo
09:11
Ayer pronostico reservado, veremos como evoluciona hoy
09:12
Pudimos verla a las 19:00 y me quedé más tranquilo; pensaba que ni llegaba. Hoy tengo visita presencial a las 18:30, pero la verdad que la noticia de vomitar por la noche ha sido un mazazo
Israel Naranjo
09:13
jode y solo puedes ir a verla a esos horarios . q putada
Cristian Lorenzo
09:15
Bueno, es también que le vaya bien a mi mujer. Pero necesita estar con terapia de fluidos y analgésicos, así que tampoco imagino que se puede estar todo el rato ahí
Israel Naranjo
09:47
compañero podrias dar una ojeada a la pull, tiene que entrar https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/12
Israel Naranjo
10:06
de micro te dijeron algo???
Cristian Lorenzo
10:07
Si, ya está confirmado. Me ha llegado lo del onboarding y tal, pero no el contrato aunque lo mandaron. Ahora lo reclamo
10:07
Ya le dije a Kike y Núria
Israel Naranjo
10:10
:cara_sonriente_con_lágrima:
Cristian Lorenzo
11:57
Te puse un comentario. Creo que los token y tal que se inyectan deben ir en las respectivas feature folder con tal vez la excepción del core (logger, por ejemplo).
11:57
Con eso, yo lo mergearía ya
Israel Naranjo
11:59
i love you , hace estas cosas si que me gustannnn
11:59
que te ha dicho esta gente
Cristian Lorenzo
12:00
Nada, que les sabe mal, pero que se alegran por mi
Israel Naranjo
12:00
plofffff
Cristian Lorenzo
12:01
El approve final igual te lo ha de dar un compañero que no sea yo, porque me han dado malas noticias de la perra y no sé si podré trabajar o concentrarme bien mucho rato más
Israel Naranjo
12:03
oki tio
12:03
joder que mal
Cristian Lorenzo
12:05
Si. Creo que no saldrá de esta. A las 15-16 le hacen otra ecografía, pero tiene dos frentes abiertos delicados.
Israel Naranjo
12:05
los perros son muy duros , pero es un putadon tio , sal a pasear para distraerte , que le den por culo a esto
Cristian Lorenzo
12:06
Sí. Voy a hacer pepeleo y poco más
Israel Naranjo
12:12
lo de q te piras se lo has comentado al equipo???
Cristian Lorenzo
12:12
Ahora lo haré. Estoy firmando los papeles, que me acaban de llegar
Israel Naranjo
12:13
me da un monton de pena mamonasooo
Israel Naranjo
09:56
No has venido a la ofi ?
Cristian Lorenzo
09:56
No
Israel Naranjo
09:56
:sudor:
Cristian Lorenzo
09:58
No creo que vaya ya hasta el dia 30 a devolver las cosas. Estoy muy triste y afectado.
Israel Naranjo
09:59
Jode
Israel Naranjo
14:48
Cristian
Cristian Lorenzo
14:49
Dime
Israel Naranjo
14:49
una pregunta sobre Traveler create entities and services with pre existing passengers
14:50
como manejar cuendo tenemos un traveler con datos , que no recordamos un comentario q hiciste
Cristian Lorenzo
14:52
Tengo un knowledge transfer de 3 a 4. Podemos verlo luego. En esencia es,
Se tiene que ir llamando en bloques workbench -> updatable items -> updated items,
Se compara lo que hay con lo que buscamos e itentamos meterlo todo en la menor cantidad de ciclos como el del punto 1,
Esto es así porque solo se permite una creación de entidad por pasajero y llamada.
:marca_de_verificación_blanca:
1
Israel Naranjo
14:52
con nosotros?
Cristian Lorenzo
14:53
Burton/Sauron
Israel Naranjo
15:00
he estado hablando antes co kike , david ... para comentarles lo de juntar los 2 fes traveler-information y el non-air-modify . no me he sabido explicar y me dicen de refactorizar uno y luego ir añadiendo el otro .
Cristian Lorenzo
15:03
No lo veo, es un cambio delicado, sobretodo lo de pasar a usar JSON API, y poder tener los dos servicios temporalmente es una ganancia
Israel Naranjo
15:04
ya, no supe como dirijir la conversacion .
Cristian Lorenzo
15:05
Necesitas que te apoye el equipo en esto, yo estoy con un pie fuera y no me comería el marrón- si sale regular- o el merito- si sale bien- de la acción
15:06
Y eso le quita fuerza a que pueda ayudarte con eso
Israel Naranjo
15:07
normal tio . joder note pires mamon
Israel Naranjo
10:43
no das la meeting de las 11 ?
Cristian Lorenzo
10:44
He dicho que en ese dominio realmente no aporto, porque no he tenido desarrollo o debugging nunca.
:+1:
1
Cristian Lorenzo
09:14
Buenas, vi los docs de confluence y tiene pinta de irse de las manos de nuevo. Personalmente, no recomiendo en absoluto que emplees next.js. Tiene convenciones muy fuertes y requerimientos de hosting importantes- así como temas de estabilidad- que si no vas a emplear RSC no merecen en absoluto esa complejidad tan fuerte que añade.
Israel Naranjo
09:39
como comento que ya esta montado en varios proyectos , he incluye su propio DI
09:39
perdon comento rodrigo
Cristian Lorenzo
09:39
En ninguno con ese objetivo. En demos con SSR sí, pero no en un servicio para la SPA
Israel Naranjo
09:40
no lo tengo a mano pero creo que hay 2 fes si no me equivoco
09:41
tu que ecomiendas no usarlo y la DI InversifyJS ??
Cristian Lorenzo
09:42
Para como es el servicio, patron singleton sencillo con constructores. También tienes nestjs que incorpora todos esos patrones y sí se usa en un servico (fes-terminal).
Israel Naranjo
09:45
tu que me recomendarias, en total , como dices es cierto sobredimensione
Cristian Lorenzo
09:48
express, o habla con Sauron a ver si vovlerian a usar nestjs o se arrepienten de ello/no volverían a hacerlo.
09:50
Y luego pensar en los patrones nuevos progresivamente. Por ejemplo el circuit breaker es algo único en SPC e intuyo que para muchos en su experiencia. Si se mete algo realmente nuevo, hay que saber explicar por qué vale la pena y lucharlo. Si no, es mejor delegarlo a futuro.
Israel Naranjo
09:51
el otro dia me reuni con esta gente kike rodrigo y manrique https://github.com/Tvlport-SPC/fes-cloud-scripts/blob/main/src/security/security.module.ts
Cristian Lorenzo
09:51
Vale, eso es nestjs
09:52
Así que lo han usado al menos en dos servicios, por lo que les parece bien
Israel Naranjo
09:53
por eso pense par aque meter otra libreria de DI si ya tenemos eso . lo elavorare un poco mas . para hablarlo con kike . si me pones comentarios , te lo agradeceria asi por lo menos me oriento
Cristian Lorenzo
09:54
Sí, pero en el doc https://travelport.atlassian.net/wiki/spaces/EV/pages/667353254/Next.js+Implementation?at[…]g1N2FlMjZjNjY0MGQiLCJwIjoiY29uZmx1ZW5jZS1jaGF0cy1pbnQifQ, hablas de nextjs
Next.js Implementation
Owned by: isnave | 2 people viewed
More actions...
Añadido por Confluence Cloud
09:54
Pero el framework que usan es nestjs
09:54
nest de nido
09:54
https://nestjs.com/
nestjs.comnestjs.com
NestJS - A progressive Node.js framework
NestJS is a framework for building efficient, scalable Node.js web applications. It uses modern JavaScript, is built with TypeScript and combines elements of OOP (Object Oriented Programming), FP (Functional Programming), and FRP (Functional Reactive Programming).
Israel Naranjo
12:38
me podrias poner criticas constructivas en la documentacion ??? porfavorrrrrrrrrrr
Cristian Lorenzo
12:42
En cual?
12:42
El de Next? Yo creo que si el que tenias de referencia lo de rodrigo, es otro framework
Israel Naranjo
12:42
del technical design que puse queestavamos hablando
12:43
en general todo lo que puse que opinas
Cristian Lorenzo
13:29
Vale, ya lo revisé
13:29
Voy a descansar que en media hora tengo sesiones
Israel Naranjo
14:30
muchas gracias
Israel Naranjo
11:40
cristian , desde donde se hace el deploy de dev1 a rumiko
11:40
???
Cristian Lorenzo
11:41
No hay algo tipo gitops dev?
11:41
Con las Actions?
11:41
No tengo el favorito a mano, que baje a tomar un café
Israel Naranjo
11:42
jurrr
Cristian Lorenzo
12:24
Alguna de aquí https://github.com/Tvlport-SPC/spcplatform-gitops-spc-nonprod/actions/workflows/deploy_service.yml?
:manos_levantadas:
1
Israel Naranjo
11:18
Cristian una duda , los ssr de DOB , se ponen en el chd o en el primer pasajjero
Cristian Lorenzo
11:19
Cada pasajero puede tener sus ssrs
Israel Naranjo
11:19
de DOB
Cristian Lorenzo
11:19
Sí. Cada pasajero que tenga minimum sec docs tendrá un par de SSRs
Israel Naranjo
11:20
pero para diferenciar de los ssr de la edad de los chd e infant
11:20
que son los que ponemos a manos por ejemplo por terminal
Cristian Lorenzo
11:20
Yo nunca he puesto a mano un SSR de edad/genero. Name remarks sí
11:20
Cada uno el suyo
Israel Naranjo
11:22
hostia, es que el DOB de un chd se esta cascando ennel segundo pasajero ADT + ADT +CHD
Cristian Lorenzo
11:22
No sé lo bastante de eso. Puede ser que el niño esté asignado a un adulto también, pero ni idea de los internos
Israel Naranjo
11:24
te suena que se pueda identificar si un ssr de dob oara los chd o inf .
11:24
sorry te estoy dando la barrila a 1 dia de salir disparado
Cristian Lorenzo
11:25
Nacho seguro que sabe más de eso. Como se reflejan en el PNR en formato texto no lo controlo bien
Israel Naranjo
11:26
na investigare que me esta volviendo loco ... Mamon no te pires
Cristian Lorenzo
11:27
Al final es en Glòries, más cerca que en Diagonal Mar
Israel Naranjo
11:28
q cabrito a tiro piedra de mi casa