Adrian Lopez
:prohibido_el_paso:
Mensajes
Añadir canvas
Archivos ,
CanvasListaCarpeta
Cargando historial...
Israel Naranjo
13:15
adri
13:15
si puedes pillate mi bug , estoy de medicos
13:15
y es una tonteria q tu ya sabes
Adrian Lopez
:prohibido_el_paso: 13:16
oki, sin problema! creo que el mio ya tengo localizado donde realmente tendre que hacer los edits asi que puedo pillarlo sin problema
Israel Naranjo
13:17
este son
plugins/plugin-traveler-information-gui/src/publicComponents/Email/EmailType.tsx
plugins/plugin-traveler-information-gui/src/publicComponents/Email/EmailAddress.tsx
13:18
voy a comer q en un rato debo volver a salir
Adrian Lopez
:prohibido_el_paso: 15:34
isra, se me esta complicando un poco la tarea mia como ya tenia pensado pero conforme me la quite de encima me pongo con la tuya un sec
15:35
(al parecer es tocar varios plugins y estoy intentando ver si puedo hacer el minimo impacto)
Israel Naranjo
16:47
nada acabo de llegar me lio con ella
Israel Naranjo
08:21
que te dijeron al final
08:22
te fueron a recoger ? saliste tarde
Adrian Lopez
:prohibido_el_paso: 08:22
que estaba todo igual pero me cambiaron la medicacion para los dolores, estuve alli como 6 horas
08:23
y mi padre me vino a recojer asi que al final llegue a casa reventado jajajaj
08:24
bueno y que evite hacer esfuerzos en el baño, que eso empeora el asunto siempre
Israel Naranjo
08:45
tranki tio .
Israel Naranjo
16:48
Adri en packages/retain-review-and-confirm-state/src/interfaces.ts el Offer lo pusiste de -> @pos-web/pnr-mocks-factory me dicen q lo pusi te tu
Adrian Lopez
:prohibido_el_paso: 16:49
sep, lo puse yo
Israel Naranjo
16:49
y eso te parece que esta bien una interfaz pillar de un mock
Adrian Lopez
:prohibido_el_paso: 16:51
o sea yo puse el de pnr mocks factory que era donde me comentaron que tenia que ponerlo
16:51
no sabia si se tenia que poner en otro sitio o exportarse de alguna manera en concreto
Israel Naranjo
16:51
yo creo q eesta mal
16:52
creo ojo , supongo por l oque mire en los repos lo usas par los test
16:52
pero par adefinir una interfaz par el codigo
Adrian Lopez
:prohibido_el_paso: 16:52
hmmmmm cierto
16:54
o sea me parece un poco extraño pero no se si realmente se podria meter en el mismo lado que el complete pnr o ponerlo directamente en el retain review and confirm state ya que es un objeto que solo se usa ahi
16:54
(o sea que solo se usa en teoria para consolidar los tipos de passengers que acepta el carrier)
Israel Naranjo
16:55
podria ponerse en unsitio comun pero no usar de un mock
16:55
eso me parece
Adrian Lopez
:prohibido_el_paso: 17:00
vale si, la cosa sera poner un offer en donde el retain-review-and-confirm-state
17:00
estas ahora con el retain-review-and-confirm o esta david?
Israel Naranjo
17:00
no me comentan que te lo diga
17:01
si modifican la factoria de test espera q te fallen los test no codigo q se ejecute
Adrian Lopez
:prohibido_el_paso: 17:02
oki, voy a apañar eso entonces
17:02
mucho animo con el medico a ver si es leve y no te tienen tanto tiempo ahi esperando
Adrian Lopez
:prohibido_el_paso: 14:41
cuando estes listo dime y llamo!
Israel Naranjo
14:41
dame 1 min
Adrian Lopez
:prohibido_el_paso: 14:41
oki!
Israel Naranjo
14:43
cuando kieras
Se ha realizado una junta 14:43
Tú y Adrian Lopez participasteis en la junta durante 2 min.
Israel Naranjo
14:51
pasme la pr de los rowbuilder (editado)
Adrian Lopez
:prohibido_el_paso: 14:51
voy!
Israel Naranjo
14:52
haver si rula y se lo meto en el ojete
Adrian Lopez
:prohibido_el_paso: 14:52
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/544
#544 [DRAFT] feat(plugin-hotel-traveler-details): add rowbuilder compatibility on …
…hotel-traveler-details
Description
This PR updates the Hotel Traveler plugin to use the RowBuilder component on its Email and Telephone fields.
Motivation and Context
PR Checklist
Please check if your PR fulfills the following requirements:
• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
• Refactoring (no functional changes, no api changes)
Mostrar más
Tvlport-SPC/pos-web-traveler-details | 2 dic. 2024 | Añadido por GitHub
14:52
es esta de aqui
14:52
en teoria a traves del sonarqube deberias poder ver que grupo de tests fallanb
Israel Naranjo
14:53
bien o lo tiro en local
Adrian Lopez
:prohibido_el_paso: 14:53
o en local tambien
14:53
el test que suele fallar es uno de NewHotel noseque
Adrian Lopez
:prohibido_el_paso: 10:20
isra, cuando tengas lo tuyo listo dame un toque y te hago llamada entonces
Israel Naranjo
10:31
:+1::tono-de-piel-5:
Adrian Lopez
:prohibido_el_paso: 15:35
isra una duda
15:35
image.png
image.png
15:35
es normal que en el plugin sell hayan tests que fallan asi aleatoriamente?
15:35
o sea que algunos pasen, tiro workflow otra vez, y fallan otros
Israel Naranjo
16:10
en local tambien ?
16:11
tardandela cuenta y timeout
Adrian Lopez
:prohibido_el_paso: 16:11
ahora me ha pasado en la PRç
16:11
es muy aleatorio si
16:12
y si, todos los tests que fallan son por timeouts
Israel Naranjo
16:12
quese ha puesto que puede ralentiz el flujo ??
Adrian Lopez
:prohibido_el_paso: 16:15
lo unico que he añadido es esto, para saber cual es el workflow en el que esta actualmente
image.png
image.png
Israel Naranjo
16:21
const getDefaultWorkflow = useMemo(() => {
return useMakeSelectors('getDefaultWorkflow');
}, []);
Adrian Lopez
:prohibido_el_paso: 16:23
me salta un "React Hook "useMakeSelectors" cannot be called inside a callback. React Hooks must be called in a React function component or a custom React Hook function."
16:24
tambien he probado el test en main, sin tocar nada, y no me pasan tampoco estando en main
Israel Naranjo
16:24
joed q movid
Adrian Lopez
:prohibido_el_paso: 16:26
sisisi
Adrian Lopez
:prohibido_el_paso: 13:56
buenas isra, he mirado lo de los errores usando codigo antiguo y he llegado a la conclusion que salian dos errores igualmente
13:57
eso usando el tunel con las versiones antiguas antes de modificar el codigo mio
image.png
image.png
13:58
(voy a ver si es por algo mas esto)
Israel Naranjo
15:46
eso es mmal
Adrian Lopez
:prohibido_el_paso: 15:47
sep, estoy mirando a ver exactamente antes como funcionaba lo del error outlet y ver que cambio ha hecho que suceda esto
Israel Naranjo
15:48
se abre un fix
Adrian Lopez
:prohibido_el_paso: 15:48
no si en teoria ahora con lo de las PRs ya se arregla
Israel Naranjo
15:49
con lo del if es un apaño pero lo suiyo es corregoirlo
Adrian Lopez
:prohibido_el_paso: 15:50
sep, eso es lo que estoy mirando a ver el porque pasa, porque añadiendo una action sola no deberia tener que ver nada con que se duplique
Israel Naranjo
15:50
voy a mirar
Adrian Lopez
:prohibido_el_paso: 15:52
plugin-sell: https://github.com/Tvlport-SPC/pos-web-plugin-sell/commit/bc4059803f6cd401254f99e33eef5c72a7a18a84
traveler-details: https://github.com/Tvlport-SPC/pos-web-traveler-details/commit/d4e621483e3a8e5b20dbc3cb54374a20101bebd4
15:52
por si quieres compararlo, estos son los commits pre-errores
Adrian Lopez
:prohibido_el_paso: 16:02
vale
16:02
acabo de verlo
16:02
y ahora entiendo el porque
16:02
image.png
image.png
16:02
este show error outlet solo hace trigger cuando se hace un submit
16:03
y ese submit solo se ejecuta si estas en el review and confirm propio
16:03
en cualquier otro, no se hace submit por ese boton, se hace submit por botones propios que se añaden en la interfaz
16:08
por lo tanto, creo que como ahora ese showErrorOutlet no existe, se mostraba siempre (dos veces fuera de review and confirm); se ha hecho el cambio de que se muestre el banner solo al hacer click en ese boton que solo se muestra en rnc a que se muestre el banner solo si el workflow es rnc
Israel Naranjo
16:15
tiene sentido
Adrian Lopez
:prohibido_el_paso: 16:31
isra, una duda
16:31
image.png
image.png
16:31
a esta pantalla exactamente como se llega?
16:32
he visto que el problema del overlap no es por traducciones si no por css del componente y he visto que se puede reproducir en otros workflows pero en este en concreto como se llega?
Israel Naranjo
16:53
llegaste a corregir lo de los comboss ?
Adrian Lopez
:prohibido_el_paso: 16:54
esta chema con ello ahora, voy a preguntar
Israel Naranjo
16:54
pufff chema :muriéndose_de_risa:
Israel Naranjo
08:57
cuando tengas un momento hablamos
Adrian Lopez
:prohibido_el_paso: 09:06
oki, desayuno y te digo!
Adrian Lopez
:prohibido_el_paso: 09:17
cuando estes listo llama que ya estoy preparado
Israel Naranjo
09:19
oki te llamo ?
Adrian Lopez
:prohibido_el_paso: 09:19
yepp
09:19
dale
Adrian Lopez
:prohibido_el_paso: 14:16
Isra, podrias darle la review a esto que asi ya lo mergeo cuando este listo? https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/578
#578 fix(sell): handle replace window layout to set fullscreen if on RnC layout
Description
This PR adds a handleReducedAction that changes the mode from Collapsed to Fullscreen if the user is on the Review and Confirm layout.
Motivation and Context
This patch has been done in order to fix a Backward-exclusive bug where the window is Collapsed when going into seats and never returned into Full Screen when coming back into R&C which is not compatible with Mini-views.
PR Checklist
Please check if your PR fulfills the following requirements:
• The commit message follows our guidelines
• Tests for the changes have been added (for bug fixes / features)
• Docs have been added / updated (for bug fixes / features)
Types of changes
What kind of change does this PR introduce?
• Bug fix
• New… Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 29 ene. | Añadido por GitHub
Israel Naranjo
14:29
te falta resolver los comentarios
14:30
ya te di el approve
Adrian Lopez
:prohibido_el_paso: 14:30
deberia avisar a cristian para que los acepte o al cambiarlos yo ya puedo aceptarlos?
Israel Naranjo
14:30
seria lo suyo
Adrian Lopez
:prohibido_el_paso: 14:31
okey! voy a ello entonces
Israel Naranjo
14:31
:+1::tono-de-piel-5:
Israel Naranjo
10:47
dale socio https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/571
:marca_de_verificación_blanca:
1
Adrian Lopez
:prohibido_el_paso: 10:49
hecho!
Israel Naranjo
10:50
done , ahora lo probare en dev1 cuando esté
Israel Naranjo
16:14
https://travelport.atlassian.net/wiki/spaces/NTAC/pages/44107289/NDC+Internal+PCC+Test+Data
NDC Internal PCC Test Data
Purpose
Owned by: Bob Hunter | 10 comments | 396 people viewed
More actions...
Añadido por Confluence Cloud
Israel Naranjo
17:46
Tu has hecho alguna vez lo de los dependabot , cierto ?
Adrian Lopez
:prohibido_el_paso: 17:46
sep, un par de veces solo pero sep
Israel Naranjo
17:47
oki mañana me lo explicas , 0 veces
Adrian Lopez
:prohibido_el_paso: 17:47
JAJAJAJ
Israel Naranjo
17:47
:sonrisa_con_sudor:
Adrian Lopez
:prohibido_el_paso: 17:47
no problem! mañana te digo
Israel Naranjo
17:47
:gratitude-thank-you:
Israel Naranjo
09:16
Adri donde se mira lo del dependaBot ?
Adrian Lopez
:prohibido_el_paso: 09:17
en teoria en las PRs mismas
09:17
pera que busco una
09:19
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/584
#584 build(deps): bump the dependencies group across 1 directory with 101 updates
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.
---
Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
Mostrar más
Labels
dependencies
Tvlport-SPC/pos-web-traveler-details | 11 feb. | Añadido por GitHub
09:19
esta por ejemplo
09:19
si todos los tests pasan, adelante, se puede mergear
09:19
(como si fuera una pr normal)
09:19
si no, hay que ver porqueç
09:19
una de las causas puede ser la subida de ciertos componentes
09:20
por ejemplo, esta parece ser que es del i18next
image.png
image.png
09:20
tambien hay que mirar el dependabot.yml que hay en .github
09:21
image.png
image.png
09:21
este por ejemplo le falta añadir dos excepcionjes
09:21
donde el ignore, hay que reemplazarlo por esto
09:21
ignore:
- dependency-name: '*'
update-types: ['version-update:semver-major']
- dependency-name: 'node-fetch' # devDependency only, if upgraded all tests fail
- dependency-name: '@pos-web/plugin-windows-grid' # contains hidden breaking changes
09:21
el nodefetch porque si se sube petan los tests
09:22
y el plugin windows grid porque hay un breaking change que hace el pete
Israel Naranjo
09:22
venga lo miro par entenderlo
Adrian Lopez
:prohibido_el_paso: 09:22
image.png
image.png
09:22
dependabot ya te da una rama para que trabajes con ella asi que
09:23
a partir de ahi puedes ir haciendo las modificaciones que necesites
Israel Naranjo
09:23
hostia q bueno
Adrian Lopez
:prohibido_el_paso: 09:23
(que sera muy seguramente bajar lo que veas que ha de bajarse
09:23
si son componentes de fuera, yo personalmente los pondria en el dependabot.yml ignore
Adrian Lopez
:prohibido_el_paso: 14:07
oye isra, tu sabes sobre servicios y tal?
Israel Naranjo
14:08
dime haber si te puedo ayudar
Adrian Lopez
:prohibido_el_paso: 14:10
image.png
image.png
14:10
este es el session manager que he movido las configmaps de un lado a otro
14:10
la cosa es que esta fallando por algo del nodejs
14:11
y no es el unico fes que esta fallando
14:11
cualquier fes de los ultimos dos dias falla
Israel Naranjo
14:12
la telemetria esta cascando
Adrian Lopez
:prohibido_el_paso: 14:12
estoy viendo que hay algo del tema aqui
image.png
image.png
14:12
por lo tanto, no se si deberia subir ese fes-session-manager a qa1 hasta que no solucionen esto
Israel Naranjo
14:13
otia hasta q no se corrija y cambiar el paquete ... (editado)
Adrian Lopez
:prohibido_el_paso: 14:14
sep, es que me parece bastante arriesgado subirlo a qa1
14:15
cuando hay el pete ya en dev1 y es una cosa generalizada
Israel Naranjo
14:15
yo me esperaria
14:15
haber si lo corrigen rapido
Adrian Lopez
:prohibido_el_paso: 14:16
eso espero porque esto lleva desde la tira...
Israel Naranjo
14:17
quien lo lleva scott ?
Adrian Lopez
:prohibido_el_paso: 14:17
esto parece que hizo el reporte scott si, no se quien lo lleva exactamente
Israel Naranjo
14:18
para estar al tanto..
Adrian Lopez
:prohibido_el_paso: 14:19
he preguntado a un contacto de burton a ver que me diga mas o menos
14:19
que tambien estaban implicados en ello por lo que supongo y entiendo
Israel Naranjo
14:20
le estoy preguntando al Manrrique
Adrian Lopez
:prohibido_el_paso: 14:22
okioki
Israel Naranjo
14:26
a las 3 han montado una pequeña call para explicar lo que hay que hacer le dije que nos metiese
Adrian Lopez
:prohibido_el_paso: 14:26
ok!
14:27
asi al menos pues sabemos de que va el asunto
Israel Naranjo
14:27
exacto , haber si nos meten
Adrian Lopez
:prohibido_el_paso: 14:28
vale, acabo de recibir el correo
Israel Naranjo
14:29
y yo perfect
Adrian Lopez
:prohibido_el_paso: 12:08
isra, voy a ponerme con un defect ahora, si necesitas algo de ayuda y estoy disponible dime y te ayudo
Israel Naranjo
12:09
oki , ya te digo
Adrian Lopez
:prohibido_el_paso: 10:10
isra, quieres hacer la llamada ahora y te intento ayudar un poco?
Israel Naranjo
10:16
si si dame 5 minutos
Israel Naranjo
10:25
ya
10:26
te llamo
10:26
?
Adrian Lopez
:prohibido_el_paso: 10:26
llama llama
Se ha realizado una junta 10:27
Tú y Adrian Lopez participasteis en la junta durante 20 min.
1 respuesta
hace 8 mesesVer hilo
Adrian Lopez
:prohibido_el_paso: 09:41
si quieres puedes cojerlo para acabar de comprobar, no logro poder ponder desde el R&C el membership number asi que no se exactamente que me falta
09:41
si que en todo caso lo que ha dicho cristian
09:41
image.png
image.png
09:41
las opciones es solo esto por lo que se podria borrar supongo
Israel Naranjo
09:42
quitar se deberia poder se lo puse la ultima vez por que estaba a piñon
09:42
que me quede dormido me acabo de levantar
Adrian Lopez
:prohibido_el_paso: 09:42
no prob no probç
09:43
por si acaso, es que no me acaba de cuadrar, esto siempre esta en read only?
image.png
image.png
Israel Naranjo
09:43
el frag
Adrian Lopez
:prohibido_el_paso: 09:44
el frag?
Israel Naranjo
09:44
?features=EV-39673-CUSTOMER-LOYALTY-EDITION:true
Adrian Lopez
:prohibido_el_paso: 09:44
ah
09:44
vale eso era lo que me faltaba JAJAJA
09:44
pero si, pillatelo si quieres, no hay problema
Israel Naranjo
09:45
como veas yo es por avanzar
Adrian Lopez
:prohibido_el_paso: 09:46
si no estas con nada pillalo, si estas con algo ya lo cojo yo :músculo::músculo:
Israel Naranjo
09:47
lo pillo
Adrian Lopez
:prohibido_el_paso: 10:18
isra, sobre lo del ticket, estoy mirando y hay bastantes cosas a actualizar de los dependabots (y acabar de rellenar la ficha que puso nuria si puedo hacerlo), mejor te pones a hacer lo de david tu y conforme acabe lo del membership me pongo de apoyo contigo mientras hago lo del dependabot?
Israel Naranjo
10:28
cmo veas
10:28
yo soy un mandao
Adrian Lopez
:prohibido_el_paso: 10:47
mejor entonces hacemos eso, voy a ponerme con el prb y conforme este listo te aviso para hacer llamada
Adrian Lopez
:prohibido_el_paso: 14:35
Vale, ya he hecho lo del prb, quieres que hagamos llamada por si te puedo echar un cable?
Israel Naranjo
14:55
ya esta me falta q me pase el test
14:55
cuando lo suba lo revisas por si acaso
Adrian Lopez
:prohibido_el_paso: 14:56
perfect! cuando este pasame el enlace y lo miro
:+1:
1
Adrian Lopez
:prohibido_el_paso: 15:30
isra, lo he borrado yo por lo que hemos dicho en la dialy
15:30
JAJAJAJ
15:30
que al final se borraba
Israel Naranjo
15:30
tio
15:30
pero si estaba con el bug e incluye eso
15:31
me acabas de matar
Adrian Lopez
:prohibido_el_paso: 15:31
pero al final no ibas a ponerte con lo de david y yo acabar lo del prb?
Israel Naranjo
15:32
la puta ...
15:34
ùes si si quieres nos ponemos con lo de david me lie
Adrian Lopez
:prohibido_el_paso: 15:34
no pasa na hombre, tendria que haber especificado yo tambien el que asi que no worries
15:34
te llamo por aqui o por webex'
Israel Naranjo
15:34
tu mismo
Se ha realizado una junta 15:34
Tú y Adrian Lopez participasteis en la junta durante 47 min.
Israel Naranjo
11:39
Adri se debe de pagar a los abogados aparte de los 100 euros del principio
otros 100 euros mas ivo no ??
Adrian Lopez
:prohibido_el_paso: 11:39
sep, enviaron un correo hace un tiempo en el correo personal
11:39
si no creo que puedes preguntarles por correo que te pasen los datos o ir a las oficinas y que te digan como pagarlo (porque a mi me pasaron un numero especifico de referencia en el correo vamos) (editado)
Israel Naranjo
11:41
voy a mirar porque no lo encuentro
Adrian Lopez
:prohibido_el_paso: 11:41
mira en spam y si ves que no encuentras nada como Ronda, enviales un correo si eso o llamales
11:41
te busco el email o telefono si eso
11:42
cronda@cronda.coop o si no el 93 268 21 99
11:42
si ves que no lo encuentras
Israel Naranjo
11:44
gracias tio
11:44
qtal en el otro equipo
Adrian Lopez
:prohibido_el_paso: 11:46
no probs! y de momento bien, esta siendo reaprender varias cosas desde cero ya que los repos y componentes son distintos pero poco a poco
Israel Naranjo
11:47
me alegro
Adrian Lopez
:prohibido_el_paso: 11:47
por ahi como va?
Israel Naranjo
11:54
mucho jaleo mas cosas menos gente
Adrian Lopez
:prohibido_el_paso: 12:06
mas o menos igual por aqui, hay mucha cosa que hacer y con el cambio de ownerships igual
Israel Naranjo
10:51
una duda Adri, del Dependabot que se aztualiza solo los paquete de travelport o todos
Adrian Lopez
:prohibido_el_paso: 11:13
todos en teoria
11:13
a no ser que sea un breaking change pero en teoria deberia de actualizarse todos, pero no se si hay algunos que hay que bloquear como el node-fetch
11:13
porque si no rompen los tests o algo asi
Israel Naranjo
11:15
:sudor:
Israel Naranjo
15:25
Adri compi, el tema de los errores del review and confirms, recuerdas como quedo al final , lo de las columnas que agrupaban los errores colores y demas . recuerdas si hay algun figma ?
Adrian Lopez
:prohibido_el_paso: 15:26
creo que al final no recibí nada sobre el tema, o sea los error de validation es como siempre pero los warning de validation no se nada de nada
15:27
no se si sabran algo por parte de ui/ux en general
Israel Naranjo
15:28
na na , como no veo que haya lo de las columnas de errores igual dijeron de cambiarlo , no recuerdo .
Adrian Lopez
:prohibido_el_paso: 15:28
yo tampoco recuerdo nada del tema tampoco, se que se dejó muy al aire pero nada mas
Israel Naranjo
15:32
:bailarín: