un approve más? venga, que es gratis :apuntando_hacia_arriba_2:


Israel Naranjo
10:36
Una review please -> https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/601


2 respuestas
Última respuesta hace 8 mesesVer hilo


Pedro Morata Vila
12:00
pregunta tonta: Alguien me puede decir en que formularios se usa el fes-non-air-modify?


Israel Naranjo
12:09
el GBF
:+1:
1



David Vazquez
12:21
ahora mismo en el review and confirm para el telefono y email de la agencia
:+1:
1



Israel Naranjo
18:39
una pequeña pr que salio otro tema de los errores del puto GBF https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/603


Jose Martin
11:11
Pequeña PR para arreglar el comentario de Nacho de mi tarea: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/605
#605 fix(plugin-traveler-information-gui): add traveler information section bottom margin
Description
Adds a bottom margin for the traveler information to avoid this:
image
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?
Mostrar más
Tvlport-SPC/pos-web-traveler-details | 4 mar. | Añadido por GitHub
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/605



David Vazquez
16:41
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/644
#644 feat(plugin-sell): add telemetry and warning banner fro persistence
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 4 mar. | Añadido por GitHub


Cristian Lorenzo
17:46
Para revisar, vulnerabilidades https://github.com/Tvlport-SPC/pos-web-custom-keys/pull/401
#401 fix: solve boost vulnerabilities
Description
Solve boost vulnerabilities
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
Mostrar más
Comments
1
Tvlport-SPC/pos-web-custom-keys | 4 mar. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 09:15
Para revisar, lo del plugin settings panel: https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1507
#1507 feat(plugin-settings-panel): add partial settings load compatibility …
Description
This PR allows the plugin to show the new type of settings object return as commented in this PR (Tvlport-SPC/fes-plugin-settings#93). Now the plugin allows for partial settings to be loaded and shows a warning on those who aren't either with a valid schema or the user doesn't have a valid option.
This also refactors the casting of partials to avoid code repetition.
With a future update, the settings won't be returning an empty object if one single plugin returns a nonvalid value and it will warn the user that defaults are being used if the setting schema is valid but the user setting is not.
Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-43165
PR Checklist
Ple… Mostrar más
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 28 feb. | Añadido por GitHub


2 respuestas
Última respuesta hace 8 mesesVer hilo


Cristian Lorenzo
09:19
Para revisar https://github.com/Tvlport-SPC/pos-web-custom-keys/pull/403
#403 fix: solve vulnerabilities
Description
Solve vulnerabilities:
• nwsapi, uses by JSDOM as CSS select engine, has a regression with the management of the ':' char. We need to fix it to an older version until gets fixed
Motivation and Context
Boost issues
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?
Mostrar más
Tvlport-SPC/pos-web-custom-keys | 5 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
2



David Vazquez
09:41
he hecho unos cambios y se resetearon los votos, podéis volver a darle approve https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/644?
#644 feat(plugin-sell): add telemetry and warning banner fro persistence
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 4 mar. | Añadido por GitHub


4 respuestas
Última respuesta hace 8 mesesVer hilo


David Vazquez
12:34
pequeña PR con un par de cambios que hacen falta https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/607
#607 feat(plugin-traveler-information-gui): export action deleteTravelersConfirmationRequest
Description
Export action deleteTravelersConfirmationRequest to display remove travelers modal.
Remove sticky position from error banner to display correctly with the warning banner
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?
Mostrar más
Tvlport-SPC/pos-web-traveler-details | 5 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
16:08
Para revisar: https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/295
#295 fix(reservation-details-sync-engine): keep pnr traveler order in conciliated transaction
Description
JIRA task: EV-44060
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
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 5 mar. | Añadido por GitHub


Jose Martin
16:29
echadle un vistacín anda :apuntando_hacia_arriba_2:


Cristian Lorenzo
16:34
Eso lo tenemos bien controlado?
id: travelerData.passengerNumber.toString(),


Joan Perez camprubi
:eat: 16:34
justo iba a comentar eso


Cristian Lorenzo
16:34
Que cosas así la lían parda


Jose Martin
16:35
así es como hay que enviar el id para que el traveler-information no pete
16:36
eso mismo se hace en el mapper para crear el defaultValues:
16:36
image.png

image.png
16:36
es bien raro, pero así funciona bien


Cristian Lorenzo
16:37
Guay


Jose Martin
16:37
si quieres puedo meter los cambios en rumiko y hacer pruebas allí


Joan Perez camprubi
:eat: 16:37
pero entonces para que se comprueba que tenga _id?
16:37
Porque esta en el pnr?


Jose Martin
16:37
eso le pregunté a David que lo quitó en su momento, y eso causaba el pete del R&C de la semana pasada


Cristian Lorenzo
16:38
Porque el fes-traveler-information -> cs-traveler-information es raro


Joan Perez camprubi
:eat: 16:38
desde luego


Cristian Lorenzo
16:38
Y hace que no se creen travelers
16:38
Y falla el end-transact


Joan Perez camprubi
:eat: 16:38
los IDs en travelport no se le puede llamar ID


Israel Naranjo
16:38
q lio


Joan Perez camprubi
:eat: 16:38
es falso en su definicion ya de por si
16:38
xD
16:38
Como te llamas?
16:38
Me llamo 31


Israel Naranjo
16:39
:signo_de_interrogación_rojo:


Jose Martin
16:40
he reiniciado los approve, he hecho el cambio que me pedía Isra


Joan Perez camprubi
:eat: 16:40
Como has dormido hoy?
16:40
Hace sol


Israel Naranjo
16:41
:marca_de_verificación_blanca:


Jose Martin
16:42
es un poco locurote, si
16:42
super contraintuitivo


Joan Perez camprubi
:eat: 16:42
Es de 1ero de Hola Mundo


Jose Martin
18:03
alguien está haciendo cambios en el plugin-sell que vaya a mergear dentro de poco?


Adrian Lopez
:prohibido_el_paso: 18:07
nope


David Vazquez
18:08
yo
18:09
bueno no sé si dentro de poco porque al final parece que nos van a pasar un diseño para lo que estoy haciendo


Jose Martin
18:52
ok, sino ya hago PR yo subiendo la versión del paquete
18:53
mañana miramos


Adrian Lopez
:prohibido_el_paso: 10:54
Para revisar: https://travelport.atlassian.net/wiki/spaces/EV/pages/410780490/DRAFT+SPC+-+Invalid+user+settings+indicator
[DRAFT] SPC - Invalid user settings indicator
|Authors|@Adrian Lopez |
|Team|Rumiko (@Adrian Lopez @Cristian Lorenzo @David Vazquez @Inacio Baldovino @isnave @Joan Perez camprubi @Jose Martin @Nuria Trevino @Pedro Morata V...
Owned by: Adrian Lopez | 1 person viewed
More actions...
Añadido por Confluence Cloud


Jose Martin
15:02
Un cambio pequeño: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/646
#646 build(plugin-sell): update reservation-details-sync-engine
## Description
updates reservation-details-sync-engine package
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44060
• 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-plugin-sell | 6 mar. | Añadido por GitHub


Israel Naranjo
09:48
para revisar -> https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/613
:+1::tono-de-piel-2:
1



Jose Martin
10:38
Por cierto, he estado probando Cursor. El IDE este con AI y es cojonudísimo
:ojos:
1

10:39
por si a alguien le pica la curiosidad


Israel Naranjo
10:39
lo suelo usar esta chulo


Jose Martin
10:40
me ha sorprendido mucho


Cristian Lorenzo
10:54
El otro día Rodenas me enseño el Claude Code, muy bueno. Las cosas estándard y pequeñas están practicamente resueltas


Jose Martin
10:54
le echaré un vistazo también a ver que tal


Israel Naranjo
11:34
para re-revisar (corregidos los comentarios)-> https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/613


11 respuestas
Última respuesta hace 8 mesesVer hilo


Pedro Morata Vila
11:52
tengo la 212 en breve
11:53
121...


Israel Naranjo
11:53
:muriéndose_de_risa:


Israel Naranjo
14:22
@Adrián López @Pere Morata habeis subido la version de .net ? o necesitais una mano?



6 respuestas
Última respuesta hace 8 mesesVer hilo


Pedro Morata Vila
14:23
No hacía falta tocar nada
:this_splunk:
1



Adrian Lopez
:prohibido_el_paso: 15:09
Para revisar (subida de dependencias y bloqueo de subida del windows-grid por parte del dependabot):
https://github.com/Tvlport-SPC/pos-web-input-recall/pull/122
#122 build(deps): bump the dependencies group across 1 directory with 19 updates
Bumps the dependencies group with 19 updates in the / directory:
| Package | From | To |
| ------------------------------------------------------------------- | ------- | ------- |
| @pos-web/devkit-scripts | 9.0.49 | 9.0.51 |
| @pos-web/eslint-config | 5.0.49 | 5.0.51 |
| [@react-aria/focus](https://github.com/adobe/react-spectrum) | 3.19.1 | 3.20.0 |
Mostrar más
Labels
dependencies
Comments
1
Tvlport-SPC/pos-web-input-recall | 9 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Adrian Lopez
:prohibido_el_paso: 15:40
:advertencia: Para revisar, es la PR para el fix de urgencia que nos acaban de avisar por team-rumiko https://github.com/Tvlport-SPC/pos-web-agency-tools/pull/232 (editado)
#232 fix(plugin-profiles-manager-gui): add missing orion-analytics import
## Description
Priority Test System Issue where the plugin has a missing @orion/analytics import.
## 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-agency-tools | 10 mar. | Añadido por GitHub
:manos_levantadas::tono-de-piel-2:
1


1 respuesta
hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 15:56
ha respondido a un hilo de la conversación:
:advertencia: Para revisar, es la PR para el fix de urgencia que nos acaban de avisar por team-rumiko https://github.com/Tvlport-SPC/pos-web-agency-tools/pull/232
Esta el automerge activado y todos los checks pasan, con una review ya tira palante :mano_con_signo_de_aprobación:
:marca_de_verificación_blanca:
1



David Vazquez
16:12
para revisar https://github.com/Tvlport-SPC/pos-web-pnr/pull/1685
#1685 fix(plugin-pnr-gui): fix payment readonly condition on pnr retrieve success
## Description
## 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 | 10 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1

16:13
no parece que se rompa nada de lo que había antes


Cristian Lorenzo
16:24
A ver que dicen los de Miró


David Vazquez
16:32
no han dicho nada, ya está mergeado


David Vazquez
16:53
me voy a pillar esta tarea https://travelport.atlassian.net/browse/EV-44012 mientras espero a que nos pasen el diseño de la que tenía asignada
EV-44012 [CR] SPC: Drop Down of Credit Cards to contain Card Code EP for IAT…
Status: In Development
Type: Story
Assignee: David Vazquez
Priority: 2 - High
More actions...
Añadido por Jira Cloud
16:54
no recuerdo si había algo más del retain state que fuera prioritario para la primera versión y la primera demo


Cristian Lorenzo
17:01
Adelante con esa. Mañana revisamos por donde vamos y os comento lo de whatfix
:+1:
2



Pedro Morata Vila
09:40
tienen algun tipo de complejo con la longitud de las cosas?
image.png

image.png


Cristian Lorenzo
09:53
Lo malo es cuando usan distintos base path por funcionalidad en lugar de una única API gateway
09:56
Para revisar https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1526
#1526 feat(plugin-window-layout): reflect on the DOM the current layout
• Required to ease the integration of Whatfix with SPC. Part of https://travelport.atlassian.net/browse/EV-43536
## Description
Reflect in the DOM the current window layout id to ease the integration with vendors
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43536
• 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-kernel-plugins | 10 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
12:35
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/649
#649 Feat/add skeleton of air sell task
## Description
This is a follow-up of the proposal to write flows as collections of task. In this first PR, we are adding the names of task
that make a book and hold flow in a new package.
Still needs to be typed correctly, but as this is still darkly, is in the best interest and spirit of CI to keep merges daily.
## 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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 10 mar. | Añadido por GitHub


Cristian Lorenzo
15:36
Para revisar
15:36
https://github.com/Tvlport-SPC/pos-web/pull/832
#832 feat(app-kernel-whatfix): add userId to global __WHATFIX_DATA__
## Description
Add userId to global WHATFIX_DATA
## Is this change related to a Jira Task?
• Yes
• Story/task Link: https://travelport.atlassian.net/browse/EV-43441
• 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 | 11 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
2



Adrian Lopez
:prohibido_el_paso: 15:49
Para revisar: https://github.com/Tvlport-SPC/pos-web-devkit/pull/378
#378 fix(devkit-settings-kernel): add STORAGE_QUOTA_EXCEEDED service status
## Description
This PR fixes a case where the localStorage quota could be exceeded and the users wouldn't be able to get or set their settings if no localStorage was found.
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44189
• 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
Tvlport-SPC/pos-web-devkit | 11 mar. | Añadido por GitHub


Jose Martin
15:51
Esto se puede ir mirando también: https://github.com/Tvlport-SPC/fes-non-air-modify/pull/131
#131 feat(fes-non-air-modify): implement agency corporate code
## Description
JIRA task: https://travelport.atlassian.net/browse/EV-43856
## 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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
## Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Comments
1
Tvlport-SPC/fes-non-air-modify | 11 mar. | Añadido por GitHub
15:51
Lo subiré a rumiko para hacer pruebas y ver si funciona bien


Cristian Lorenzo
16:30
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/651
#651 fix(plugin-sell): remove duplicate telemetry data
## Description
Remove duplicate telemetry data
• Three properties are already provided by the metadata decorator plugin, causing confusion in our analysts.
## 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.
• My changes generate no new warnings.
Mostrar más
Tvlport-SPC/pos-web-plugin-sell | 11 mar. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 16:56
Para revisar: https://github.com/Tvlport-SPC/pos-web-devkit-integration/pull/41
#41 fix(devkit-settings-driver-service): catch localStorage exception if quotadata has exceeded
## Description
This PR fixes a case where the localStorage quota could be exceeded and the users wouldn't be able to get or set their settings if no localStorage was found.
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44189
• 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
Tvlport-SPC/pos-web-devkit-integration | 11 mar. | Añadido por GitHub


Cristian Lorenzo
17:11
PAra revisar https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/615
#615 fix(plugin-traveler-information-gui): add defensive code on error man…
## Description
Add defensive code on error management section
## Motivation and Context
Breaking code
## 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
Comments
1
Tvlport-SPC/pos-web-traveler-details | 11 mar. | Añadido por GitHub


Cristian Lorenzo
08:56
Para revisar https://github.com/Tvlport-SPC/pos-web-custom-keys/pull/410
#410 Feat/ev 32032 indent exported pkeys
## Description
Use new indented XML serializer for custom keys export.
## Motivation and Context
Improved UX, part of https://travelport.atlassian.net/browse/EV-32032
## 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
Comments
1
Tvlport-SPC/pos-web-custom-keys | 11 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Israel Naranjo
09:42
Alguien necesita una mano ???


Cristian Lorenzo
09:43
Vamos a organizar ahora un plan de trabajo para lo del testarama


Israel Naranjo
09:44
genial


Adrian Lopez
:prohibido_el_paso: 10:00
Para revisar: https://github.com/Tvlport-SPC/pos-web-devkit-integration/pull/42
#42 fix(devkit-settings-driver-service): change DIO console from test-utils to constructor-based class variable
## Description
This PR includes a hotfix that swaps the getDIO from test-utils to the one delivered by diManager, being saved into the class instead.
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44189
• 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
Tvlport-SPC/pos-web-devkit-integration | 12 mar. | Añadido por GitHub


Israel Naranjo
10:29
me pillo este https://travelport.atlassian.net/browse/EV-43854 no mangar .... matooo

EV-43854 Error handling known errors
Status: In Development
Type: Story
Assignee: isnave
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud
:sonrisa_con_sudor:
1
:+1::tono-de-piel-2:
1



2 respuestas
Última respuesta hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 10:44
esta petando dev1?
10:44
(he borrado caché y ahora no me deja entrar)


Cristian Lorenzo
10:44
Yo si puedo entrar


Jose Martin
10:46
y yo


Pedro Morata Vila
10:47
si teniais una sesion abierta, os funcionara.
10:48
si cerrais sesion, si os ha caducado y teneis que rehacerla o lo que sea.... os saldra un mensaje tal que así
10:48
image.png

image.png
10:48
(probad, por ejemplo, rumiko, que posiblemente no teniais sesion creada)


Pedro Morata Vila
11:29
rumiko is back online
11:30
y qa


Cristian Lorenzo
13:04
Para revisar https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1528
#1528 Feat/reflect current views
## Description
Reflect current views to ease the integration with Whatfix
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43536
• 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-kernel-plugins | 12 mar. | Añadido por GitHub


David Vazquez
15:34
para revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/411
#411 feat(air-payment-gui): add new credit card type and export method to get types
## Description
• Add new credit card option EasyPay to the card types list.
• Create a feature flag 'FF-DISABLED-CARD-TYPE-OPTIONS' to enable or disable specific card types options for the agent.
• Publish a method on plugin-payment-api-connection to get the list of card types instead of using the constant CARD_TYPES
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44012
• Feature Link:
Mostrar más
Tvlport-SPC/pos-web-payment | 12 mar. | Añadido por GitHub


17 respuestas
Última respuesta hace 8 mesesVer hilo


Cristian Lorenzo
16:31
Para revisar https://github.com/Tvlport-SPC/pos-web-pnr/pull/1688
#1688 feat(pnr-gui): expose current path to DOM
• Required to integrate Whatfix.
• Part of https://travelport.atlassian.net/browse/EV-43536
## Description
## 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.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-pnr | 12 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
16:38
Vale, el servicio guarda bien el corporate Id de la agencia. Pero hay algunas cosillas que mirar y decidir
16:38
pero por ahora puedo seguir avanzando
16:38
Quien irá a la oficina mañana por cierto? (yo iré)
:-1:
1



Cristian Lorenzo
16:43
Yo imagino que sí
:+1:
1



Adrian Lopez
:prohibido_el_paso: 16:45
yo también iré a la ofi mañana


Israel Naranjo
16:48
no ire tengo q ir a rehabilitacion


Joan Perez camprubi
:eat: 16:49
yo tampoco que me pierdo clase sino


David Vazquez
16:49
yo iré también


David Vazquez
18:29
ya he ampliado el coverage y resuelto el comentario, se puede volver a revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/411 (editado)
#411 feat(air-payment-gui): add new credit card type and export method to get types
## Description
• Add new credit card option EasyPay to the card types list.
• Create a feature flag 'FF-DISABLED-CARD-TYPE-OPTIONS' to enable or disable specific card types options for the agent.
• Publish a method on plugin-payment-api-connection to get the list of card types instead of using the constant CARD_TYPES
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44012
• Feature Link:
Mostrar más
Comments
1
Tvlport-SPC/pos-web-payment | 12 mar. | Añadido por GitHub


David Vazquez
15:18
ya se puede revisar de nuevo, ya pasan los tests y estan arreglados los comentarios


Jose Martin
15:30
Para revisar https://github.com/Tvlport-SPC/fes-non-air-modify/pull/132
#132 fix(fes-non-air-modify): fix service not reporting errors
## Description
The service endpoint was not reporting the errors
## 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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
## Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Tvlport-SPC/fes-non-air-modify | 13 mar. | Añadido por GitHub


Cristian Lorenzo
15:40
Para revisar https://github.com/Tvlport-SPC/pos-web/pull/838
#838 feat(app-kernel-endpoint-url): export getEndpointURL
## Description
Export getEndpointURL:
• Modified the kernel to export a default symbol getEndpointURL, which allows to get the URL of an end-point.
• Refactored the plugin to use typescript
## 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.
Mostrar más
Comments
1
Tvlport-SPC/pos-web | 13 mar. | Añadido por GitHub


Israel Naranjo
16:29
un ojo please https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/617


4 respuestas
Última respuesta hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 16:43
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/618
#618 fix(plugin-traveler-information-gui): add validation to not allow future DOB on traveler info
## Description
This PR adds a validation rule where a traveler DOB can't be a future date than the current one.
## Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-44083
## 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 | 13 mar. | Añadido por GitHub

1 respuesta
hace 8 mesesVer hilo


Israel Naranjo
16:55
una duda el dateFormat = 'YYYY-MM-DD' o deberia la region del navegador ???




27 respuestas
Última respuesta hace 8 mesesVer hilo


Israel Naranjo
17:28
como llevamos las tareas?
17:28
Los objetivos se han cubierto ???


Jose Martin
17:29
esta semana vamos sobrados


Israel Naranjo
17:29
:beso_de_corazón:


Jose Martin
17:29
si, en teoría habíamos puesto la mía del servicio que ya está hecha


Joan Perez camprubi
:eat: 17:31
nais


Israel Naranjo
17:31
La pregunta del millón :tambor_con_baquetas:


Jose Martin
17:31
si, Cristian ha confirmado de se hace flexi


Israel Naranjo
17:32
donde ?


Joan Perez camprubi
:eat: 17:32
que bien


Jose Martin
17:32
aquí a mi lado


Israel Naranjo
17:32
uuuuuuuu


Joan Perez camprubi
:eat: 17:32
pues a gozar :manos_corazón:
17:32
buen finde


Israel Naranjo
17:44
os va dev1?


Jose Martin
17:44
no, los entornos llevan caídos la últimas dos horas
17:45
lo han avisado
17:45



Naomi O'callaghan
Advance notice - Travelport+ test system network disruption
FYI @spc-tech-leads @spc-team-sdet @Jade Williams
This is starting at 3pm GMT/4pm BCN today
Mostrar más
Hilo en accessui-spc-priority-test-system-issues | 13 mar. | Ver el mensaje


Israel Naranjo
17:45
merdaaaaa
17:47
Ya van


Adrian Lopez
:prohibido_el_paso: 17:51
ha respondido a un hilo de la conversación:
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/618
dadle un vistacillo que ya me chuta el quality gate


Israel Naranjo
17:52
paymen petin
image.png

image.png


Jose Martin
17:55
si, lo está mirando David


Israel Naranjo
17:59
:fuego:


David Vazquez
18:05
pequeña PR para revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/413
#413 feat(air-payment-gui): add missing hook on package
## 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.
• My changes generate no new warnings.
Mostrar más
Tvlport-SPC/pos-web-payment | 13 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Adrian Lopez
:prohibido_el_paso: 09:10
hoy estoy algo denso ya por la mañana, alguien sabe porque no puedo poner ningun membership number en busqueda?
image.png

image.png


Israel Naranjo
09:39
@Adrián López al bug del membershipnumber si quieres lo pillo o te ayudo , por que se por dodne ira


Adrian Lopez
:prohibido_el_paso: 10:38
?features=EV-39673-CUSTOMER-LOYALTY-EDITION:true


Adrian Lopez
:prohibido_el_paso: 11:43
https://github.com/Tvlport-SPC/pos-web-custom-keys/pull/413 para revisar (cosas de dependabot, he tenido que restringir dos paquetes que rompen los tests por el breaking change, esta documentado) (editado)
#413 build(deps): bump the dependencies group across 1 directory with 28 updates
Bumps the dependencies group with 28 updates in the / directory:
| Package | From | To |
| -------------------------------------------------------------- | -------- | ------- |
| [node-fetch](https://github.com/node-fetch/node-fetch) | 3.0.0 | 3.3.2 |
| @pos-web/atlas-components-adaptors | 1.7.81 | 1.7.90 |
| @orion/i18n | 5.24.1 | 5.27.0 |
Mostrar más
Labels
dependencies, javascript
Comments
2
Tvlport-SPC/pos-web-custom-keys | 16 mar. | Añadido por GitHub


Cristian Lorenzo
11:51
El de orion igual subiendo typescript pasa


4 respuestas
Última respuesta hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 14:12
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/620
#620 fix(plugin-traveler-information-gui): remove ProgramName from its use
## Description
This PR removes the Membership Type combobox from the Airline membership numbers. This is done to both act as a fix to a bug where the component would be shown as required and as a refactor due to it being a single-element combobox (which can be omitted as it always has the same value) plus this component not being used elsewhere outside this repository.
[image](https://private-user-images.githubusercontent.com/149699153/423485188-431b5b58-4fbc-4003-ad96-c3406c3f7445.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIyMTc0MjgsIm5iZiI6MTc0MjIxNzEyOCwicGF0aCI6Ii8xNDk2OTkxNTMvNDIzNDg1MT… Mostrar más
Comments
1
Tvlport-SPC/pos-web-traveler-details | 17 mar. | Añadido por GitHub
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/620

:marca_de_verificación_blanca:
1



Cristian Lorenzo
14:44
Para revisar https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1540
#1540 feat(plugin-reference-data): add CreditCardType dataset
## Description
Add CreditCardType dataset
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 17 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1

14:44
Para revisar tambien https://github.com/Tvlport-SPC/reference-data-import/pull/84
#84 feat: add creditCardTypes dataset
## Description
Add creditCardTypes dataset:
• It points to credit card types that SPC graphical flows can support. The property isAvailable points to those that should be offered in views, so it can be used in part a a feature toggle.
## Motivation and Context
We agreed to use this instead of a feature flag + a constant to solve https://travelport.atlassian.net/browse/EV-44012
## 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)
• Package dependencies have been updated as much as possible (for minor and patche… Mostrar más
Tvlport-SPC/reference-data-import | 17 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Israel Naranjo
15:30
@aquí Alguien ha borrado el ProgramName de pos-web-traveler-details ??


2 respuestas
Última respuesta hace 8 mesesVer hilo


Cristian Lorenzo
15:34
Para revisar https://github.com/Tvlport-SPC/pos-web-reference-data/pull/77
#77 feat: add Credit Card Types dataset
## Description
Add Credit Card Types dataset
## Motivation and Context
• Card types known and supported by graphical flows.
## 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-reference-data | 17 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
15:58
Para revisar https://github.com/Tvlport-SPC/reference-data/pull/103
#103 feat: expose credit card types dataset
## Description
Expose credit card types dataset
## Motivation and Context
Required for https://travelport.atlassian.net/browse/EV-44012
## 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
Comments
1
Tvlport-SPC/reference-data | 17 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 16:54
Para revisar:
https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/652
https://github.com/Tvlport-SPC/pos-web-fare-shop/pull/1040



4 respuestas
Última respuesta hace 8 mesesVer hilo


Cristian Lorenzo
17:07
Se reinició esto https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1540
#1540 feat(plugin-reference-data): add CreditCardType dataset
## Description
Add CreditCardType dataset
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 17 mar. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 13:18
Para revisar (dependabot): https://github.com/Tvlport-SPC/pos-web-telemetry/pull/47
#47 build(deps): bump the dependencies group across 1 directory with 71 updates
Bumps the dependencies group with 71 updates in the / directory:
| Package | From | To |
| --------------------------------------------------------------------------------------------------------------------------- | -------- | ------- |
| @pos-web/devkit-scripts | 9.0.35 | 9.0.51 |
| @pos-web/eslint-config | 5.0.35 | 5.0.51 |
| [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) | 4.22.0 | 4.33.0 |
Mostrar más
Labels
dependencies, javascript
Comments
1
Tvlport-SPC/pos-web-telemetry | 16 mar. | Añadido por GitHub


Cristian Lorenzo
13:20
No hay muchos ignore?


Cristian Lorenzo
13:21
Lo apruebo pero creo que habría que darle un vistazo


6 respuestas
Última respuesta hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 16:26
Para revisar (un dependabot que ya esta listo, necesito un review, los tests pasan correctamente):
https://github.com/Tvlport-SPC/pos-web-input-recall/pull/124 (editado)
#124 build(deps): bump the dependencies group across 1 directory with 20 updates
Bumps the dependencies group with 20 updates in the / directory:
| Package | From | To |
| ------------------------------------------------------------------- | ------- | ------- |
| [lerna](https://github.com/lerna/lerna/tree/HEAD/packages/lerna) | 8.2.0 | 8.2.1 |
| [@react-aria/focus](https://github.com/adobe/react-spectrum) | 3.20.0 | 3.20.1 |
| [@react-aria/interactions](https://github.com/adobe/react-spectrum) | 3.24.0 | 3.24.1 |
Mostrar más
Labels
dependencies, javascript
Comments
1
Tvlport-SPC/pos-web-input-recall | 16 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
16:39
Se puede ir mirando mientras pasan las pipes: https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/298
#298 feat(plugin-reservation-details.gui): add agency corporate id to mapper
## Description
## 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-plugin-reservation-details | 18 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 16:40
Y hizo buuum!


Jose Martin
16:41
re-run al canto


Joan Perez camprubi
:eat: 16:49
Lo de SARA no recuerdo como era para que apareciesen los campos
16:49
Que el vuelo vaya o provenga de una isla española
16:49
bueno territorios fuera de peninsula
16:49
Hace falta alguna FF?


Jose Martin
16:54
Esto se puede mirar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/621
#621 feat(plugin-traveler-information-gui): add agency corporate code fiel…
…d in review and confirm form
## Description
[image](https://private-user-images.githubusercontent.com/112655938/424065399-e90d39a6-ca35-4bd4-9051-5ce2db9a1d59.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDIzMTM1NjksIm5iZiI6MTc0MjMxMzI2OSwicGF0aCI6Ii8xMTI2NTU5MzgvNDI0MDY1Mzk5LWU5MGQzOWE2LWNhMzUtNGJkNC05MDUxLTVjZTJkYjlhMWQ1OS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMzE4JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDMxOFQxNTU0MjlaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1iZGQ2ZmRkM2YyM2ZkMWNhMTQ3ZmE4MTA4M… Mostrar más
Tvlport-SPC/pos-web-traveler-details | 18 mar. | Añadido por GitHub
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/621

:marca_de_verificación_blanca:
2



Joan Perez camprubi
:eat: 17:01
Lo del SARA quien estuvo trabajando en ello sabe si hace falta alguna FF?
17:01
No lanza la busqueda cuando uso ADR


Jose Martin
17:15
Otra para revisar: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/657
#657 feat(plugin-sell): add agency corporate code in common requester
## Description
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43857
• 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
Tvlport-SPC/pos-web-plugin-sell | 18 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
08:27
Para revisar https://github.com/Tvlport-SPC/pos-web/pull/841
#841 build(app-kernel-endpoint-url): change how packages are compiled to d…
## Description
Change how packages are compiled to don't get errors 404
## 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 | 18 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1

08:27
En cuanto tenga un approve hago force merge que si no cualquier neuva imagen de pos-web estará cascada


Joan Perez camprubi
:eat: 09:55
Alguien me puede ayudar?

1 respuesta
hace 8 mesesVer hilo


Joan Perez camprubi
:eat: 09:55
Sabéis si hay una lista con todas las FF y lo que hacen?



22 respuestas
Última respuesta hace 8 mesesVer hilo


Cristian Lorenzo
10:04
Para revisar https://github.com/Tvlport-SPC/pos-web-custom-keys/pull/415
#415 refactor: update dependencies
## Description
Update dependencies and use getEndpointURL to remove augmentations
## Motivation and Context
Maintenance
## 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
Comments
1
Tvlport-SPC/pos-web-custom-keys | 19 mar. | Añadido por GitHub


Jose Martin
11:55
Vale, he hecho un draft del documento para poner las feature flags: https://travelport.atlassian.net/wiki/spaces/EV/pages/425460335/Rumiko+s+Feature+Flags
Rumiko's Feature Flags
|Feature Flag|Temporal Y/N|Description|Feature’s Jira Link|
|FT_EV-19477_FAVORITE_CUSTOM_SHORTCUTS||||
Owned by: Jose Martin | 1 person viewed
More actions...
Añadido por Confluence Cloud
11:56
Está abierto a sugerencias


Jose Martin
12:03
Lo he dejado en los bookmarks del canal rumiko
image.png

image.png


Joan Perez camprubi
:eat: 12:10
guay :manos_levantadas:


Jose Martin
12:18
Se puede mirar esto: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/657
#657 feat(plugin-sell): add agency corporate code in common requester
## Description
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43857
• 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-plugin-sell | 18 mar. | Añadido por GitHub
12:18
No le echéis cuenta al quality, que está un poco loco


Adrian Lopez
:prohibido_el_paso: 12:53
Para revisar: https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1541 (dependabot)
#1541 build(deps): bump @pos-web/tooltip from 5.1.2 to 5.1.3 in the internal-dependencies group across 1 directory
Bumps the internal-dependencies group with 1 update in the / directory: @pos-web/tooltip.
Updates @pos-web/tooltip from 5.1.2 to 5.1.3
[Dependabot compatibility score](https://camo.githubusercontent.com/76b76fd70700e0ef2a1fe26fd87b275f8953b843c6309c0d5d0ee55e90f58212/68747470733a2f2f646570656e6461626f742d6261646765732e6769746875626170702e636f6d2f6261646765732f636f6d7061746962696c6974795f73636f72653f646570656e64656e63792d6e616d653d40706f732d7765622f746f6f6c746970267061636b6167652d6d616e616765723d6e706d5f616e645f7961726e2670726576696f75732d76657273696f6e3d352e312e32266e65772d76657273696f6e3d352e312e33)
Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You… Mostrar más
Labels
dependencies, javascript
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 17 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
15:18
Como va lo que teníamos previsto? Alguien necesitará ayuda?


Adrian Lopez
:prohibido_el_paso: 09:32
Para revisar: https://github.com/Tvlport-SPC/pos-web-workbench/pull/114 (dependabot, pone que se necesita review de decebal pero supongo que con una mas de rumiko deberia dejar hacer el merge) (editado)
#114 build(deps): bump the dependencies group across 1 directory with 27 updates
Bumps the dependencies group with 27 updates in the / directory:
| Package | From | To |
| ------------------------------------------------------ | ------- | ------- |
| @pos-web/app-kernel-session-manager | 2.12.0 | 2.15.3 |
| @pos-web/devkit-scripts | 9.0.46 | 9.0.51 |
| @pos-web/eslint-config | 5.0.46 | 5.0.51 |
Mostrar más
Labels
dependencies
Comments
2
Tvlport-SPC/pos-web-workbench | 10 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 09:37
Lo del "dependency-name" no es raro que este dos veces la misma llave?


17 respuestas
Última respuesta hace 8 mesesVer hilo


Joan Perez camprubi
:eat: 09:38
image.png

image.png


Adrian Lopez
:prohibido_el_paso: 10:21
Para revisar: https://github.com/Tvlport-SPC/pos-web-plugin-notepad/pull/72 (dependabot otra vez)
#72 build(deps-dev): bump the dependencies group across 1 directory with 14 updates
Bumps the dependencies group with 14 updates in the / directory:
| Package | From | To |
| ------------------------------------------------------ | ------- | ------- |
| @pos-web/devkit-scripts | 9.0.49 | 9.0.51 |
| @pos-web/eslint-config | 5.0.49 | 5.0.51 |
| [node-fetch](https://github.com/node-fetch/node-fetch) | 3.0.0 | 3.3.2 |
Mostrar más
Labels
dependencies, javascript
Tvlport-SPC/pos-web-plugin-notepad | 16 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Adrian Lopez
:prohibido_el_paso: 11:21
Para revisar: https://github.com/Tvlport-SPC/pos-web-gateway/pull/88 (añadir el archivo de dependabot)
#88 ci: create dependabot.yml
## Description
## 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
Comments
1
Tvlport-SPC/pos-web-gateway | 20 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
15:41
Para revisar: https://github.com/Tvlport-SPC/fes-non-air-modify/pull/133
#133 fix(fes-non-air-modify): update entities in reservation
## Description
PR to just update entities present in the reservation.
## 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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
## Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Comments
1
Tvlport-SPC/fes-non-air-modify | 19 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 17:58
Hey @aquí
17:58
Como van los objetivos?
17:59
Hacemos flexi friday?
18:00
Yo creo que sí, no?
18:02
Lo hablamos por wasap
18:02
Que paseis buena tarde


Jose Martin
18:18
Para revisar: https://github.com/Tvlport-SPC/fes-non-air-modify/pull/134
#134 fix(fes-non-air-modify): add okta baseurl for session reservation
## Description
## 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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
## Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Tvlport-SPC/fes-non-air-modify | 20 mar. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 13:00
Para revisar (poner el city del india gst como required, y he aprovechado para arreglar un visual bug que enviaba el error a tomar viento visualmente):
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/624
#624 fix(plugin-traveler-information-gui): add requirement to city on india gst form
## Description
This PR changes the City input from the India GST form into a required input. It also fixes an error position problem.
| Before … Mostrar más
Tvlport-SPC/pos-web-traveler-details | 24 mar. | Añadido por GitHub
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/624

:marca_de_verificación_blanca:
1



Adrian Lopez
:prohibido_el_paso: 14:59
es normal que el shared components este petardeando tanto?


4 respuestas
Última respuesta hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 15:30
y bueno tambien, podeis acceder a QA1? me salta un "The Host Connection provided does not exist."



3 respuestas
Última respuesta hace 8 mesesVer hilo


Cristian Lorenzo
15:33
Para revisar https://github.com/Tvlport-SPC/pos-web-fare-shop/pull/1058
#1058 Feat/add actions as events on reprice price rules method
## Description
Add actions as events on reprice price rules method. This is required to properly manage errors on fare rules updates after price checks.
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43564
• 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-fare-shop | 24 mar. | Añadido por GitHub


Cristian Lorenzo
09:19
Para revisar https://github.com/Tvlport-SPC/pos-web-car/pull/478
#478 feat(plugin-car-ui): use reference data to retrieve default card types
# Title
## Description
Use reference data to retrieve default card types.
It used a hardcoded reference data and a non-typed Payment component, which lead to the loss of integrity.
## Is this change related to a Jira Task?
• Yes
• Jira Link: https://travelport.atlassian.net/browse/EV-XXXXX
• No
## Is this change related to a Figma Design?
• Yes https://travelport.atlassian.net/browse/EV-44463
• Figma Link: https://www.figma.com/file/XXXX
• No
## Screenshots
## PR Checklist
Please check if your PR fulfills the following requirements:
• The commit message follows our guidelines.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-car | 24 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



9 respuestas
Última respuesta hace 8 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 10:22
:luz_giratoria: Para revisar (v2 del fastpatch para el payment): https://github.com/Tvlport-SPC/pos-web-payment/pull/426 (editado)
#426 fix(payment-gui): add temporal polymorphism to accept backward compatibility
## Description
This PR adds backward compatibility with the old Credit Card interface.
## Is this change related to a Jira Task?
• Yes
• Story Link: JIRA: https://travelport.atlassian.net/browse/EV-44463
• 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
Tvlport-SPC/pos-web-payment | 25 mar. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 11:55
:luz_giratoria: Para revisar (v2.5 del fast patch para el payment, añade el fix al credit card On File y el input que se muestra vacío si esta como read-only):
https://github.com/Tvlport-SPC/pos-web-payment/pull/428
#428 fix(payment-gui): add remaining backward compatibility values
## Description
This PR adds the second part of the fast patch in order to allow backward compatibility with the old type of Credit Card.
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44463
• 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
Tvlport-SPC/pos-web-payment | 25 mar. | Añadido por GitHub


Cristian Lorenzo
12:35
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-traveler-information/pull/272
#272 feat(plugin-traveler-information): remove dependecy on traveler-details
## Description
Remove dependency on traveler-details
## Motivation and Context
Unblock updates
## 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
Comments
1
Tvlport-SPC/pos-web-plugin-traveler-information | 25 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
13:09
PAra revisar https://github.com/Tvlport-SPC/pos-web-plugin-traveler-information/pull/273
#273 feat(plugin-traveler-information): remove deprecated dependency
## Description
remove deprecated dependency
## 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-plugin-traveler-information | 25 mar. | Añadido por GitHub


Jose Martin
16:15
Para revisar: https://github.com/Tvlport-SPC/fes-non-air-modify/pull/136
#136 fix(fes-non-air-modify): report commns layer errors
## Description
## 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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
## Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Comments
1
Tvlport-SPC/fes-non-air-modify | 25 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
2



Cristian Lorenzo
18:18
Btw, David, todos OK con lo del nuevo plugin, así que adelante
:+1:
1



Cristian Lorenzo
09:45
Para revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/429
#429 fix(payment-gui): accept cards from on file, and from dataset
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-payment | 25 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
10:53
pequeña PR para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/627
#627 fix(plugin-traveler-information-gui): do not allow special characters in traveler street address field
## Description
Avoid service error by adding an inline validation not permitting special characters in the street address field:
[image](https://private-user-images.githubusercontent.com/112655938/426949732-abbbd110-c171-4866-811c-2ab2fdf61eb0.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDI5ODMxMDIsIm5iZiI6MTc0Mjk4MjgwMiwicGF0aCI6Ii8xMTI2NTU5MzgvNDI2OTQ5NzMyLWFiYmJkMTEwLWMxNzEtNDg2Ni04MTFjLTJhYjJmZGY2MWViMC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMzI2JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDMyNlQwOTUzM… Mostrar más
Tvlport-SPC/pos-web-traveler-details | 26 mar. | Añadido por GitHub (3 kB)
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/627



5 respuestas
Última respuesta hace 8 mesesVer hilo


David Vazquez
11:22
haciendo unas pruebas con Nuria hemos visto que el banner de error en el GBF no aparece cuando hay un error


David Vazquez
11:22
no había una tarea para esto? o me suena que ya se comentó hace un par de semanas

1 respuesta
hace 8 mesesVer hilo


Pedro Morata Vila
11:46
Estoy probando esto en PP
https://travelport.atlassian.net/browse/EV-44238
Entiendo que como está etiquetado para la 4.20, no está aun disponible en PP, no?
EV-44238 Hold button is disabled if at least one offer is ticketed
Status: Accepted
Type: Defect
Assignee: Joan Perez camprubi
Priority: 2 - High
More actions...
Añadido por Jira Cloud


Jose Martin
13:15
Alguien consigue ver que ha fallado aquí? https://github.com/Tvlport-SPC/pos-web-traveler-details/actions/runs/14081846172/job/39437845858?pr=627
13:15
no me deja desplegar


Joan Perez camprubi
:eat: 13:21
timeout de jest


Jose Martin
13:21
merci


Cristian Lorenzo
15:21
Para revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/430
#430 refactor(payment-gui): enhance compatiblity with card reference data
## Description
Enhance compatibility with card reference data:
• Refactored selectors to reduce number of objects creation,
• getPaymentCreditCardPropertyValue accepts both shapes of data,
• Enforced correct memoizing patterns.
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44463
• Feature Link:
Mostrar más
Comments
1
Tvlport-SPC/pos-web-payment | 26 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1

15:22
El coverage da igual, es parte de la deuda técnica que ya cotiene el repositorio


Cristian Lorenzo
16:00
Join from the webinar link
https://tvpt.webex.com/tvpt/j.php?MTID=m5f95a68a927cb52819a756c7b546c5de
Join by the webinar number
Webinar number (access code): 2631 898 6405
Cisco Webex SiteCisco Webex Site
Meet virtually with Cisco Webex. Anytime, anywhere, on any device.
Simple, modern video meetings for everyone on the world's most popular and trusted collaboration platform. (15 kB)
https://tvpt.webex.com/tvpt/j.php?MTID=m5f95a68a927cb52819a756c7b546c5de



Cristian Lorenzo
16:12
Fix rápido que la cague al declarar una dependencia https://github.com/Tvlport-SPC/pos-web-payment/pull/431
#431 fix(air-payment-gui): solve missing caret in plugin dependency
## 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.
• My changes generate no new warnings.
Mostrar más
Tvlport-SPC/pos-web-payment | 26 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
17:12
Para revisar: https://github.com/Tvlport-SPC/pos-web-agency-tools/pull/235
#235 fix(plugin-profiles-manager-gui): avoid overflow hidding when closing the profiles results section
## Description
Jira defect: https://travelport.atlassian.net/browse/EV-44509
## 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
Comments
1
Tvlport-SPC/pos-web-agency-tools | 26 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
2



David Vazquez
17:54
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/307
#307 feat(pricing-api-connection): create new plugin for api connections
• Create new plugin-pricing-api-connection
• Move getOfferPricing method from plugin-sell to the new plugin and export it
• Move interfaces required by the previous method
## Description
## Is this change related to a Jira Task?
• Yes
• Story/task Link: https://travelport.atlassian.net/browse/EV-35892
• Feature Link: https://travelport.atlassian.net/browse/EV-35890
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-air-price | 26 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 18:00
Te falta un 1% de coverage
18:00
Que tocada de webex


David Vazquez
18:08
ahora ya pasa


David Vazquez
09:47
podéis volver a aprobar la PR? tuve que hacer un par de cambios


Jose Martin
10:08
image.png

image.png


Joan Perez camprubi
:eat: 10:23
https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/668


Joan Perez camprubi
:eat: 10:23
A revisar, fix del prepopulation del sara names


5 respuestas
Última respuesta hace 8 mesesVer hilo


Jose Martin
11:54
Vale, ya he corregido lo que hacía fallar los tests: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/627
#627 fix(plugin-traveler-information-gui): do not allow special characters in traveler street address field
## Description
Avoid service error by adding an inline validation not permitting special characters in the street address field:
[image](https://private-user-images.githubusercontent.com/112655938/427484850-13df63d7-b7b9-442f-8ce5-f342caacc928.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3NDMwNzMxNjYsIm5iZiI6MTc0MzA3Mjg2NiwicGF0aCI6Ii8xMTI2NTU5MzgvNDI3NDg0ODUwLTEzZGY2M2Q3LWI3YjktNDQyZi04Y2U1LWYzNDJjYWFjYzkyOC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjUwMzI3JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI1MDMyN1QxMDU0M… Mostrar más
Tvlport-SPC/pos-web-traveler-details | 26 mar. | Añadido por GitHub (7 kB)
https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/627



Adrian Lopez
:prohibido_el_paso: 12:17
Para revisar: https://travelport.atlassian.net/wiki/spaces/EV/pages/437911701/SPIKE+Investigate+Implementation+of+Price+with+Upsells
[SPIKE] Investigate Implementation of Price with Upsells
Owned by: Adrian Lopez | 2 people viewed
More actions...
Añadido por Confluence Cloud


5 respuestas
Última respuesta hace 8 mesesVer hilo


David Vazquez
13:05
se me había olvidado meter una cosa en el package.json del nuevo plugin, para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/308
#308 feat(pricing-api-connection): update package orion utils
## Description
## 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
Tvlport-SPC/pos-web-plugin-air-price | 27 mar. | Añadido por GitHub


Pedro Morata Vila
13:53
esta oferta ya es mas digna....
Me alegra que se salven 7 personas de la quema


Pedro Morata Vila
14:11
acabo de pasarme a octopus energy. me ha llegado una factura de 180€ 2 meses
14:12
estaba pagando el kwh a 0.32!!!!!


Jose Martin
14:15
si, me pasa exactamente igual


Jose Martin
14:16
si te llegas a esperar te hubiéramos pasado el código de referidos


12 respuestas
Última respuesta hace 7 mesesVer hilo


Israel Naranjo
14:16
eso me paso com iberdrola


Jose Martin
14:16
lo mío era con Endesa
14:16
exactamente ese precio (0.32)
14:16
una pasada


Cristian Lorenzo
14:30
La oferta es muy buena


Cristian Lorenzo
15:24
Para revisar, se esta perdiendo en el sync engine el mail type cuando se añade un agency por terminal
15:24
https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/301
#301 fix(reservation-details-sync-engine): maintain email type on createCh…
## Description
maintain email type on CreateCheckpointFromPnr
## Motivation and Context
Agency emails from PNR were not correctly restored and all were mapped to Traveler emails.
## 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-plugin-reservation-details | 27 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1

15:24
Tambien falla el telefono, pero lo hago después de que entre este


Joan Perez camprubi
:eat: 15:47
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/668


Cristian Lorenzo
16:05
El fix de los telefonos: https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/303
#303 fix(reservation-details-sync-engine): correctly derive phone type fro…
## Description
Correctly derive phone type from PNR data on createCheckpointFromPnr:
• Recovered role,
• Recovered city
## Motivation and Context
Bug fixing
## 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
Mostrar más
Tvlport-SPC/pos-web-plugin-reservation-details | 27 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
16:22
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/669
#669 refactor: update sync-engine related dependencies
• It fixes issues with emails, and telephones.
## Description
Update dependencies, required to fix some issues with sync-engine
## 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.
• My changes generate no new warnings.
Mostrar más
Tvlport-SPC/pos-web-plugin-sell | 27 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
10:29
Para revisar de nuevo https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/669, he arreglado temas de timeouts en tests https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/669
#669 refactor: update sync-engine related dependencies
• It fixes issues with emails, and telephones.
## Description
Update dependencies, required to fix some issues with sync-engine
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 27 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Pedro Morata Vila
12:47
bueno, me voy a poner un pinchar una **** con un palo, para un programa de deteccio/prevencion de cancer culorectal o algo asi
:zurullo:
1
:boca_abierta:
1



Israel Naranjo
12:52
?????


Joan Perez camprubi
:eat: 12:52
comorr


Pedro Morata Vila
12:58
que me han enviado carta para que me haga una prueba de sangre en heces.


Joan Perez camprubi
:eat: 12:58
ah


Pedro Morata Vila
12:58
porque en terrassa tienen un plan de prevencion de cancer colorectal


Joan Perez camprubi
:eat: 12:58
pensaba que te iban a meter un palo por el culo


Pedro Morata Vila
12:59
jajajajjajaja


Israel Naranjo
13:02
:cara_con_ceja_levantada:


Adrian Lopez
:prohibido_el_paso: 11:28
Para revisar (segunda tanda del spike): https://travelport.atlassian.net/wiki/spaces/EV/pages/437911701/SPIKE+Investigate+Implementation+of+Price+with+Upsells
[SPIKE] Investigate Implementation of Price with Upsells
Owned by: Adrian Lopez | 2 comments | 6 people viewed
More actions...
Añadido por Confluence Cloud


Pedro Morata Vila
12:34
ei chicos, dejadme que os comparta el ejercicio que me han propuesto para el puesto de .net.
Lo haré esta tarde en cuanto cierre aqui, el que quiera unirse para pair-mob programming y ver como pinta el tema rest en .net está mas que invitado.
# Technical Exercise
# Duration: 90-120 mins
# Note: Do what you think is important - it's not about completing every detail - it's more important to demonstrate good practice and clean code.
# Exercise
## Objective
Create an API in ASP.NET Core (C#) that implements a simple CRUD API with a clean application architecture. It should be easy to run and provide an API client (like Swagger) to demonstrate it's functionality.
## Details
- The project doesn't need authentication or authorization, only a simple CRUD application to register people with details: name, surname, passport number and phone.
- The name, surname and passport number will be mandatory. The validation for for the passport number must check format and avoid invalid passport numbers being collected.
- Passport number format expected will begin with the letter P or L and will be followed by another letter and 7 numbers.
- The persistence should be done with with EF Core.
- Use, whenever you can, the asynchronous methods of EF for the operations with the database.
## Tehnical Notes
Unit testing is important. Code should be testable. For now, demonstrate the tests you consider necessary to cover the core logic of the application.
We want to see your coding style, desing patterns, approach to testing and application architecture.
Feel free to make decisions on implementation, just provide a reason for doing so in a readme.
Finally, outline next steps or anything you didn't finish in a readme.
## Deliver
Share the Github/Git repo with the Visual Studio Solution. We want to see your commits.
12:37
Mi idea es dejar el tema required al motor de base de datos (entityframework tiene ya cosas sencillas para eso)
la validacion de pasaporte dejarla al servicio.
El unit testing deberia cubrir que el startup del servicio funciona, que la valildacion del usuario funciona y que el result devuelve (como idea, un objeto message, con el result y una coleccion de mensajes de error en caso necesario.)
En vuestra opinion, separariais el servicio, los modelos de entity framework, el dbcontext y el testing en proiyectos separados? o eso solo va a comerme tiempo?


Israel Naranjo
12:38
sip
12:38
separa
12:39
al menos aunquue no sean proyectos qwue sea en carpeetas
12:39
image.png

image.png


Pedro Morata Vila
12:40
ese chatgpt bueno! jejejejje
Yo estoy probancolo en claude.


Israel Naranjo
12:40
no no
12:40
desgraciado
12:40
prueba el claude code es una pasada


Cristian Lorenzo
15:58
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/305
#305 feat(plugin-reservation-details-gui): export selector factory to get …
## Description
Export selector factory to get restricted default values for identity
## Motivation and Context
Required for simpliefied ancillaries view
## 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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 31 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
2



Pedro Morata Vila
20:52
@aquí @Cristian Lorenzo alguien sabe como darle visibilidad a un usuario a un repo?
20:53
a este repositorio https://github.com/Pedro-MorataVila_tvlport/HumanManagment necesito darle visibilidad al usuario TravelportKevinG


Cristian Lorenzo
21:29
Igual en config


Cristian Lorenzo
09:17
Este bug es importante, aqyer lo reportaron en la reunión de sync sell


Cristian Lorenzo
09:17
https://travelport.atlassian.net/browse/EV-44573
EV-44573 NDC- 1G- Customer reporting NDC plugin - issue with NDC where it do…
Status: To Do
Type: Problem
Assignee: Unassigned
Priority: 2 - High
More actions...
Añadido por Jira Cloud



9 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
11:07
Para revisar https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/628
#628 feat(plugin-traveler-information-gui): add traveler names only form
## Description
Add traveler names only form
## Motivation and Context
Part of the features to streamline the ancillaries buy experience.
## 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
Comments
1
Tvlport-SPC/pos-web-traveler-details | 2 abr. | Añadido por GitHub


Jose Martin
11:42
Para revisar: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/676
#676 fix(plugin-sell): fix hold submit being blocked when adding fop
## Description
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44526
• 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
Tvlport-SPC/pos-web-plugin-sell | 2 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
13:30
dev1 anda caput, no?


Jose Martin
14:10
si
14:10
404


Jose Martin
18:06
Pera revisar, he quitado un modal que ya no se quiere usar: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/676
#676 fix(plugin-sell): fix hold submit being blocked when adding fop
## Description
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44526
• 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-plugin-sell | 2 abr. | Añadido por GitHub


David Vazquez
19:04
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/310
#310 feat(pricing-api-connection): create new redux lifecycle
## Description
## 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-plugin-air-price | 2 abr. | Añadido por GitHub


3 respuestas
Última respuesta hace 7 mesesVer hilo


Jose Martin
09:46
os funciona dev1?


Joan Perez camprubi
:eat: 09:47
no


Jose Martin
09:47
joder
09:47
DNS no?


Joan Perez camprubi
:eat: 09:47
nidea


Jose Martin
09:47
image.png

image.png


Joan Perez camprubi
:eat: 09:47
ah no
09:47
eso sera la vpn
09:47
a mi me da otro error
09:48
404 de nginx


Jose Martin
09:48
si, por eso preguntaba para probar otra ciudad


Adrian Lopez
:prohibido_el_paso: 09:49
yo estoy en la ofi
09:49
y peta
09:49
estoy subiendo rumiko para hacer las pruebas (editado)


Cristian Lorenzo
09:51
Ya lo reportaron ayer a la tarde
09:52
https://teamtravelport.slack.com/archives/C05M80RDPM3/p1743595876740709



Arnau Fugarolas
Dev1 is not working, is raising 404 when loading (editado)
image.png

image.png
Hilo en accessui-spc-priority-test-system-issues | 2 abr. | Ver el mensaje


Cristian Lorenzo
14:18
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/307
#307 feat(plugin-reservation-details-gui): allow register of required fiel…
## Description
Allow register of required fields observables
## Motivation and Context
Reversion of dependencies.
## 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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 3 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Jose Martin
17:56
Para revisar: https://github.com/Tvlport-SPC/fes-non-air-modify/pull/138
#138 fix(fes-non-air-modify): add entity id to update request body
## Description
## 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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
## Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Tvlport-SPC/fes-non-air-modify | 3 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1

17:57
Esto añade los ids a las requests de update para el email y el phone


Cristian Lorenzo
18:11
Para revisar, a ver si pasa la pipeline https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/632
#632 feat(plugin-traveler-information-gui): add form page to add travelers
## Description
Add form page to add travelers
## Motivation and Context
Required to integrate ancillaries
## 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 | 3 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
2



4 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
10:23
Necesita approve de nuevo que quedó stale https://github.com/Tvlport-SPC/pos-web-car/pull/478
#478 feat(plugin-car-ui): use reference data to retrieve default card types
# Title
## Description
Use reference data to retrieve default card types.
It used a hardcoded reference data and a non-typed Payment component, which lead to the loss of integrity.
## Is this change related to a Jira Task?
• Yes
• Jira Link: https://travelport.atlassian.net/browse/EV-XXXXX
• No
## Is this change related to a Figma Design?
• Yes https://travelport.atlassian.net/browse/EV-44463
• Figma Link: https://www.figma.com/file/XXXX
• No
## Screenshots
## PR Checklist
Please check if your PR fulfills the following requirements:
• The commit message follows our guidelines.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-car | 24 mar. | Añadido por GitHub
:marca_de_verificación_blanca:
1

10:27
Lo del OB-Fees, habrá que cambiar el servicio para que mande esa cabecera
10:27
Que bueno, es una crutrada


Israel Naranjo
10:34
@Cristian Lorenzo el spike del Adri se puede aceptar ya ?


Cristian Lorenzo
10:37
Sí. A partir de ahí podremos crear las tareas? O necesitamos más?


Israel Naranjo
10:38
se podrian crear ya , no ?


Cristian Lorenzo
10:39
Mañana en el refinement, con Maryam, que así pilla el contexto de la tarea
:+1:
2



David Vazquez
10:47
esta semana estoy de shield @Cristian Lorenzo, me pongo con lo del ob-fees que comentaron en el chat de webex? no hay tarea o defect creado para eso no?


Cristian Lorenzo
10:48
Sí, pongamos con eso. Está relacionado con los bugs de refresh payment. No sé si se uso ya alguno para eso la semana pasada.
10:48
Que se pueda continuar con el mismo
10:51
El plugin pricing api connection está caput en dev1, falta un import de getStore declarado


David Vazquez
10:55
ok


David Vazquez
11:13
un approve https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/313
#313 feat(pricing-api-connection): add getstorehelper import
## Description
## 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-plugin-air-price | 7 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
2



Adrian Lopez
:prohibido_el_paso: 11:25
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/631
#631 fix(plugin-traveler-details): rework DOB validation to force past dates
## Description
This PR changes the behavior of DoB to reallow the date to use the YYYY-MM-DD format that is sent into fes-traveler-information and force past dates onto the component (both coming from the PNR and the ones used by the DatePicker). It also fixes the Calendar component to not open by default on a current or future date.
Note
We're currently awaiting from UX to add a warning on RnC for INF and CHD users that do not meet the IATA age requirements, as finishing an NDC workflow with an invalid date ends up with a breaking error.
## Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-44573
## PR Checklist
Please check if your PR fulfills the following requirements:
• The… Mostrar más
Comments
1
Tvlport-SPC/pos-web-traveler-details | 3 abr. | Añadido por GitHub


6 respuestas
Última respuesta hace 7 mesesVer hilo


Jose Martin
15:50
He levantado rumiko (y funcionaba bien). He deployado manifiesto de plugins y servicios, y he copiado los FF de dev1 a rumiko...

1 respuesta
hace 7 mesesVer hilo


Jose Martin
15:50
y se me queda así ahora:
image.png

image.png
15:50
igual me falta algo que no recuerdo que haga falta


Adrian Lopez
:prohibido_el_paso: 16:07
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/634 (India GST, el regex que no estaba bien puesto, he añadido tests para asegurar que no se modifica el resultado del join) (editado)
#634 fix(plugin-traveler-information-gui): fix GST india regex validation to allow space characters
## Description
This PR fixes a regression on the GST regex validation and adds a test to ensure this is properly checked.
## Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-44715
## 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
Comments
1
Tvlport-SPC/pos-web-traveler-details | 7 abr. | Añadido por GitHub


Cristian Lorenzo
16:17
Lo de la cabecera OBFees del reprice y tal, hay que añadirlo al diseño técnico de OB-Fees (editado)
:+1:
1



Adrian Lopez
:prohibido_el_paso: 17:21
Para mañana cuando se haga daily, lo comento por aquí por si acaso pero estas dos tareas de JIRA ya estan ambas en QA1, uno es un PRB y el otro es un bug de la India GST, ambos solucionados y con los tests pertinentes.
https://travelport.atlassian.net/browse/EV-44573 (India)
https://travelport.atlassian.net/browse/EV-44715 (PRB)
Ya con estos dos estarian todas mis tareas listas respecto al carryover del sprint pasado (y lo de la india que es un bug que ha entrado hoy), si hubiera cualquier cosa dadme un toque igualmente tanto si me veis por la ofi como si es por Slack! :manos_levantadas::manos_levantadas::manos_levantadas: (editado)
EV-44573 NDC- 1G- Customer reporting NDC plugin - issue with NDC where it do…
Status: QA in Progress
Type: Problem
Assignee: Adrian Lopez
Priority: 2 - High
More actions...
Añadido por Jira Cloud
EV-44715 India GST - Address not allowing spaces
Status: In Acceptance
Type: Defect
Assignee: Adrian Lopez
Priority: 2 - High
More actions...
Añadido por Jira Cloud
:manos_levantadas:
1



Maryam Feizabad
18:07
se ha unido a rumiko_developers por invitación de Cristian Lorenzo
. Además, Irene Foz Almagro
se unió con invitación.


Pedro Morata Vila
10:34
recordais el libro que recomendó Cristian?
Para los que prefieren las islas caribeñas y las patas de palo
10:35
System Design Interview – An Insider's Guide Volume 1 and 2 ByteByteGo ( 2022) Alex Xu, Sahn Lam : ByteByteGo : Free Download, Borrow, and Streaming : Internet Archive
(estando en internet archive, sepuede considerar pirata?)
Internet ArchiveInternet Archive
System Design Interview – An Insider's Guide Volume 1 and 2 ByteByteGo ( 2022) Alex Xu, Sahn Lam : ByteByteGo : Free Download, Borrow, and Streaming : Internet Archive
System Design Interview – An Insider's Guide Volume ByteByteGo 1 and 2 (4 kB)
https://archive.org/details/system-design-interview-an-insiders-guide-volume-2-ByteByteGo-2022-alex-xu-sahn-lam-z-lib.io/System%20Design%20Interview%20%E2%80%93%20An%20Insider%27s%20Guide%20Volume%202%20-%20Byte%20Code%20LLC%20%282022%29%20-%20Alex%20Xu%2C%20Sahn%20Lam%20%28Z-Lib.io%29.pdf

:+1:
1

10:35
estan los 2 volumenes


Joan Perez camprubi
:eat: 10:35
ah mira
10:35
en internet se encuentra todo ya bes


Jose Martin
10:36
epub o pdf?


Joan Perez camprubi
:eat: 10:36
yo los compre ya


Pedro Morata Vila
10:36
pdf


Jose Martin
10:36
cachis


Pedro Morata Vila
10:36
ya
10:36
si alguien lo tieene en epub, que rule


David Vazquez
12:59
para revisar https://github.com/Tvlport-SPC/fes-pricing/pull/214
#214 feat(fes-pricing): update header request for repricereservation
## Description
As part of a conversation, this PR updates the header request for repriceReservationController.
curl --location 'https://nt-airreprice-offers-ntac-do-11-qab.apps.ocp-b.zu2.nonprod.travelport.io/reservation-domain-offer/OFFER_REPRICE_DOMAIN/v11_0/reservationworkbench/ef6fa902-6e2c-494a-b423-07b5a32a8015/offers/buildfromcatalogofferings' \ --header 'Accept: application/json' \ --header 'Content-Type: application/json' \ --header 'E2ETrackingID: 93ec115e-a4ce-4a46-ba3a-cc16a1b607c6' \ --header 'Accept-Version: 11' \ --header 'Content-Version: 11' \ --header 'TVP-Feature-Set: EnhancedAirChangeWorkflow' \ --header 'OBFees: test' \ --data '{ "OfferQueryBuildFromCatalogOfferings": { "P… Mostrar más
Comments
1
Tvlport-SPC/fes-pricing | 8 abr. | Añadido por GitHub


David Vazquez
13:07
he actualizado el confluence con lo de las cabeceras


Israel Naranjo
13:08
@Cristian Lorenzo te vienes a comer ??


Cristian Lorenzo
13:08
Sí


Israel Naranjo
13:08
oki


Irene Foz Almagro
16:08
Os paso dos PRs del depenbot:
build(deps-dev): bump the dependencies group across 1 directory with 45 updates by dependabot[bot] · Pull Request #53 · Tvlport-SPC/pos-web-telemetry
build(deps): bump the dependencies group across 1 directory with 29 updates by dependabot[bot] · Pull Request #120 · Tvlport-SPC/pos-web-workbench
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 16:18
Para revisar https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/630
Me faltó añadirle FF, ahora la vuelvo a pasar (editado)
:manos_levantadas:
1



2 respuestas
Última respuesta hace 7 mesesVer hilo


Maryam Feizabad
16:20
Chicos, tenemos los Feature Flags del desarrollo de Rumiko guardados en algún lado?



4 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
17:50
Para revisar, a ver si desbloqueamos el update del traveler-details: https://github.com/Tvlport-SPC/pos-web-fare-shop/pull/1074
#1074 test: disable side-effect of prefetching dataset on default testWith
## Description
disable side-effect of prefetching dataset on default testWith. This will prevent open handlers with calls made between tests. Can be overriden with,
testWith().configure(({ diManager, pluginsManagerProvider }) => {
diManager.useDelegate('__DISABLE_FARE_SHOP_PRE_FETCH__', () => {
return false;
});
pluginsManagerProvider.allowInjection('__DISABLE_FARE_SHOP_PRE_FETCH__');
});
## Is this change related to a Jira Task?
• Yes
Mostrar más
Tvlport-SPC/pos-web-fare-shop | 8 abr. | Añadido por GitHub


Cristian Lorenzo
08:22
Esta es la PR que comenté en el canal general https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/626
#626 refactor(plugin-hotel-traveler-details): use new reference data set o…
## Description
use new reference data set of available cards
## 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
Comments
1
Tvlport-SPC/pos-web-traveler-details | 24 mar. | Añadido por GitHub
:ojos:
1




14 respuestas
Última respuesta hace 7 mesesVer hilo


Joan Perez camprubi
:eat: 09:05
Para revisar de nuevo https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/630

1 respuesta
hace 7 mesesVer hilo


Cristian Lorenzo
09:17
Para revisar https://github.com/Tvlport-SPC/pos-web-air-reservation/pull/156
#156 fix(plugin-air-reservation-gui): change translation
## Description
Change translation of public fare tooltip
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44303
• 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-air-reservation | 9 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Irene Foz Almagro
10:58
Small PR actualizar dependencias :cara_ligeramente_sonriente: build(deps): bump the dependencies group across 1 directory with 8 updates by dependabot[bot] · Pull Request #98 · Tvlport-SPC/fes-plugin-settings
:marca_de_verificación_blanca:
1



Jose Martin
11:39
A quien creéis que podría contactar para pedir ayuda con postman?


2 respuestas
Última respuesta hace 7 mesesVer hilo


Joan Perez camprubi
:eat: 12:30
Se puede revisar https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/630
He subido los cambios de los comentarios, si lo veis bien los podeis resolver @Cristian Lorenzo @Maryam
:marca_de_verificación_blanca:
1



Cristian Lorenzo
12:39
Para revisar, cambios en docs https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/637
#637 docs(plugin-traveler-information-gui): add information about AddAirTr…
## Description
Add information about AddAirTravelersNameIdentifiersPage component
## Motivation and Context
Better DX for developers.
## 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
Comments
1
Tvlport-SPC/pos-web-traveler-details | 9 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
2



Cristian Lorenzo
13:19
Está jodido lo del agency... Voy a intentar acabar pronto después de comer lo de ancilliaries y si hace falta y no me escribe Kike para empezar a pillar cosas de Burton, lo miramos


Jose Martin
13:20
vale
13:20
estoy probando hacer el delete a ver si también peta
13:20
para probar una alternativa


Israel Naranjo
13:23
podriais echarle un ojo a esta PR de momento quedan algunas cosas por mirar , https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/4
:ojos:
1



2 respuestas
Última respuesta hace 7 mesesVer hilo


Irene Foz Almagro
15:44
Tengo una PR por aqui de mantenimiento: build: migrate repository to use biome and update dependencies by Irene-FozAlmagro_tvlport · Pull Request #114 · Tvlport-SPC/fes-custom-commands
Si quereis más info sobre esto aqui pusieron un post: https://teamtravelport.slack.com/archives/C05LTNE5PEJ/p1743685354387169



Rodrigo Ortega
Hi #accessui-spc-tech-sync,
As part of our ongoing efforts to upgrade our tooling, I have created a blog post detailing the steps to transition from ESLint and Prettier to Biome.
https://travelport.atlassian.net/wiki/spaces/EV/blog/2025/04/02/459833350/Streamlining+Code+Quality+Transitioning+from+ESLint+Prettier+to+Biome
Publicado en accessui-spc-tech-sync | 3 abr. | Ver el mensaje
:marca_de_verificación_blanca:
2



Maryam Feizabad
08:27
This Pr will fix the issue Scott raised in our channel:
https://github.com/Tvlport-SPC/pos-web-pnr/pull/1702
#1702 fix(plugin-pnr-similar-name-list-gui): remove z-index flight search
## Description
Fix the Z-index issue with the Table header in the booking search result
## 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 | 9 abr. | Añadido por GitHub
https://github.com/Tvlport-SPC/pos-web-pnr/pull/1702

:+1::tono-de-piel-2:
1
:marca_de_verificación_blanca:
2



Irene Foz Almagro
11:15
PR de mantemiento en el fes-plugin-settings build: migrate repository to use biome and update dependencies by Irene-FozAlmagro_tvlport · Pull Request #99 · Tvlport-SPC/fes-plugin-settings
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 11:52
Alguien más que este trabajando en esta feature?
Le puse esta FF RT-EV-44339-USE-NEW-OFFER-PRICING-METHOD pero esta abierto a sugerencias!
EV-35890 TP+ NDC - Include Reprice when the FOP is changed to calculate new …
Status: Implementing
Type: Epic
Assignee: Patricia Salgado
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud
:+1::tono-de-piel-2:
1



David Vazquez
15:31
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/315
#315 feat(pricing-api-connection): implement hook for selectPricing from offers
…ffers
## Description
## 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
Tvlport-SPC/pos-web-plugin-air-price | 9 abr. | Añadido por GitHub
:+1::tono-de-piel-2:
1



David Vazquez
17:59
ya he arreglado los comentarios, se puede revisar de nuevo


Irene Foz Almagro
09:08
Me falta likes de Rumiko, aún Maryam no cuenta como Rumiko :sonrisa: build: migrate repository to use biome and update dependencies by Irene-FozAlmagro_tvlport · Pull Request #99 · Tvlport-SPC/fes-plugin-settings
:marca_de_verificación_blanca:
1
:sonrisa_con_sudor:
1



Eduardo Vazquez
10:28
se ha unido a rumiko_developers por invitación de Maryam Feizabad
.


Joan Perez camprubi
:eat: 15:08
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/317
:marca_de_verificación_blanca:
1



2 respuestas
Última respuesta hace 7 mesesVer hilo


Irene Foz Almagro
08:33
Os paso algunas PRs de dependencias (aún no tengo poderes de Rumiko en estos plguins/servicios) :sonrisa: :
build(deps-dev): bump the dependencies group across 1 directory with 4 updates by dependabot[bot] · Pull Request #77 · Tvlport-SPC/pos-web-plugin-notepad
build(deps-dev): bump the dependencies group across 1 directory with 9 updates by dependabot[bot] · Pull Request #54 · Tvlport-SPC/pos-web-telemetry
build(deps): bump the dependencies group across 1 directory with 14 updates by dependabot[bot] · Pull Request #239 · Tvlport-SPC/pos-web-agency-tools
build(deps-dev): bump the dependencies group across 1 directory with 5 updates by dependabot[bot] · Pull Request #121 · Tvlport-SPC/pos-web-workbench
build(deps): bump the dependencies group across 1 directory with 2 updates by dependabot[bot] · Pull Request #100 · Tvlport-SPC/fes-plugin-settings
build(deps): bump the dependencies group across 1 directory with 5 updates by dependabot[bot] · Pull Request #141 · Tvlport-SPC/fes-non-air-modify
build(deps): bump the dependencies group across 1 directory with 4 updates by dependabot[bot] · Pull Request #86 · Tvlport-SPC/fes-profiles-manager
:marca_de_verificación_blanca:
3


1 respuesta
hace 7 mesesVer hilo


Cristian Lorenzo
08:49
Estuve dándole a sell caña para lo de ancilliaries, y una tarea de mantenimiento que podemos ir haciendo y que tiene valor en reducción de tiempo de CI es empezar a quitar plugin-connection-selector de los plugins de Rumiko, en particular de los no gráficos
08:50
Ya casi tengo sell, espero acabar hoy mientras espero la oportunidad de fijar nuevas prioridades con kike


Cristian Lorenzo
09:40
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/685
#685 feat(plugin-sell): remove dependency from connection-selector
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 14 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Irene Foz Almagro
09:56
Os paso otra PR de mantenimiento, he tenido que hacer un commit que pedía dos depedencias de orion build(deps-dev): bump the dependencies group across 1 directory with 7 updates by dependabot[bot] · Pull Request #638 · Tvlport-SPC/pos-web-traveler-details
:marca_de_verificación_blanca:
2



David Vazquez
10:12
le podéis volver a dar approve https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/315? Merci
#315 feat(pricing-api-connection): implement hook for selectPricing from offers
Implement hooks which will be provided with additional cache management properties
## Is this change related to a Jira Task?
• Yes
• Story/task Link: https://travelport.atlassian.net/browse/EV-44340
• Feature Link: https://travelport.atlassian.net/browse/EV-35890
• 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-plugin-air-price | 9 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



David Vazquez
10:50
se resetearon los votos, le podéis volver a dar approve?
:marca_de_verificación_blanca:
2



Irene Foz Almagro
11:18
Otra PR por aqui de mantenimiento :cara_ligeramente_sonriente: build(deps-dev): bump the dependencies group across 1 directory with 25 updates by dependabot[bot] · Pull Request #311 · Tvlport-SPC/pos-web-plugin-reservation-details
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 11:41
Esta es la PR que os comentaba en la daily https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/682
@Maryam (editado)
:+1:
1



Joan Perez camprubi
:eat: 12:11
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/320
:marca_de_verificación_blanca:
2



2 respuestas
Última respuesta hace 7 mesesVer hilo


Jose Martin
18:13
Para revisar: https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/323
#323 feat(plugin-pricing-api-connection): add price refresh telemetry
## Description
## Is this change related to a Jira Task?
• Yes
• Story/task Link: https://travelport.atlassian.net/browse/EV-44342
• Feature Link: https://travelport.atlassian.net/browse/EV-35890
• 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-plugin-air-price | 14 abr. | Añadido por GitHub


Jose Martin
18:32
queda alguien por aquí para aprobarme la PR?
:+1:
2



Maryam Feizabad
12:22
Kindly review:
https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/687


Joan Perez camprubi
:eat: 15:06
Donde creeis que es mejor testear esto?
https://cscan.travelport.com/component_measures?metric=new_coverage&view=list&pullRequest=682&id=pos-web-plugin-sell
No hay tests unitarios para esas funciones




19 respuestas
Última respuesta hace 7 mesesVer hilo


Irene Foz Almagro
15:44
PR para migrar el biome en notepad build: migrate repository to use biome by Irene-FozAlmagro_tvlport · Pull Request #78 · Tvlport-SPC/pos-web-plugin-notepad


David Vazquez
10:16
para revisar, PR para crear un container en el payment e inyectar componentes desde el sell https://github.com/Tvlport-SPC/pos-web-payment/pull/433 (editado)
#433 feat(air-payment-gui): create container to inject components to air payment form
## Description
Create and export new "airPaymentPanelContainer" to inject components to the air-payment form from other plugins to avoid circular dependencies.
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-payment | 15 abr. | Añadido por GitHub



4 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
12:48
To review: https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1558
#1558 refactor: remove prop-types
## Description
Remove props types to reduce testing noise:
• Reduce noise on test in exchange for static analysis guarantees.
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 15 abr. | Añadido por GitHub


Cristian Lorenzo
12:59
Para el que haga mantenimiento, puede usar este script para actualizar las devDependencies internas de repos grandes:
import { glob } from 'glob';
import { readFileSync, writeFileSync } from 'node:fs';

const packages = await glob(['plugins/**/package.json', 'packages/**/package.json']);

const internalPaths: [string, any][] = [];
const innerPackages = new Set();

for (const pkg of packages) {
if (pkg.search('node_modules') === -1) {
const parsed = JSON.parse(readFileSync(pkg, { encoding: 'utf-8' }));
internalPaths.push([pkg, parsed]);
innerPackages.add(parsed.name);
}
}

for (const [url, pkg] of internalPaths) {
for (const devDependency of Object.keys(pkg.devDependencies || {})) {
if (innerPackages.has(devDependency) && devDependency !== pkg.name) {
pkg.devDependencies[devDependency] = 'workspace:*';
}
}

writeFileSync(url, JSON.stringify(pkg, null, 2), { encoding: 'utf-8' });
}
:+1::tono-de-piel-2:
1




5 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
13:43
Esta es la que casca? https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/682
#682 feat(plugin-sell): use new get offer pricing ported method from plugi…
…n pricing api connection
## Description
use new get offer pricing ported method from plugin pricing api connection
## Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-44339
• 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-plugin-sell | 10 abr. | Añadido por GitHub


14 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
13:44
Si es, luego de comer tengo una hora libre para mirarla


Irene Foz Almagro
14:21
Yo tenia esta de mantenimiento (perdón son muchos ficheros :desconcertado: )build: migrate repository to use biome by Irene-FozAlmagro_tvlport · Pull Request #428 · Tvlport-SPC/pos-web-custom-keys pero me falla la build tengo que acabarlo aún


Irene Foz Almagro
15:46
Quick PR Feature/update dependencies by Irene-FozAlmagro_tvlport · Pull Request #79 · Tvlport-SPC/pos-web-plugin-notepad
:marca_de_verificación_blanca:
1



4 respuestas
Última respuesta hace 7 mesesVer hilo


Eduardo Vazquez
17:41
It'd be great if I could get some feedback on this PR: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/689
First time working on plugin-sell so I'm open to nitpicks! (editado)


2 respuestas
Última respuesta hace 7 mesesVer hilo


David Vazquez
19:04
Aquí dejo el draft de la integración porque hasta el lunes 28 estoy de vacaciones https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/690
Quería dejarlo lo más completo posible pero haciendo pruebas me he encontrado con más problemas, así que aún le queda.
Cosas a destacar, aunque muchas de ellas las he dejado comentadas en el código:
Hay que hablar con la gente que hizo el fes-pricing porque al parecer cuando recuperas una reserva y navegas a IssueTicket no pilla la información necesaria (offerId, productsId, offeringSelection) para hacer la request del getPrice porque esta la pilla del getInformationSell y no existe, esta información parece que solo existe y está disponible cuando creas una reserva desde el principio. Hay que hablar con ellos y ver si está información se puede obtener de otro lado o que hace falta. He probado sacar el offerId con otro selector del pnr-air-itinerary-gui pero parece que peta la request con este error, así que no sé, hay que preguntarles.
{
"errors": [
{
"code": 20777,
"message": "Shop response can't be found in the cache"
}
],
"name": "TravelportError",
"id": "75325631-1783-4712-9374-5302e43eca15",
"source": "fes-pricing",
"operation": "POST_PRICING",
"type": "REQUEST",
"status": 424,
"message": "Shop response can't be found in the cache",
"workflowId": "a4a2af9b-3780-4b5f-990a-47c33545be5a"
}
Hablándolo con Cristian al final inyectaremos el componente en el container solo cuando sea preET de momento, hasta que se pueda solucionar lo anterior en el flujo postET.
Segunda cosa, el hook ejecutará el método getOfferPricing para la forma de pago CREDIT_CARD cuando los campos de forma de pago, tipo de tarjeta y número de la tarjeta estén completos, ya que esos valores son necesarios para poder hacer correctamente la request de getPrice.
Tercera tarea crear el test para el componente que inyectaremos.
Cuarta tarea sacar y pasar los datos necesarios para la telemetría (está comentado en el código)
(editado)
#690 feat(plugin-sell): integrate the hook to display obfees and warnings
## 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.
• My changes generate no new warnings.
Mostrar más
Tvlport-SPC/pos-web-plugin-sell | 16 abr. | Añadido por GitHub
:+1::+1::tono-de-piel-2:
2



2 respuestas
Última respuesta hace 7 mesesVer hilo


Joan Perez camprubi
:eat: 11:09
Ya petó la pipeline en la nueva rama de pruebas https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/688/, a partir de este commit https://github.com/Tvlport-SPC/pos-web-plugin-sell/compare/1f5cfbd4d793f5503da4635d0c1eef424452b0d2..2caa469d92ad0259549bbddea5c385d5da349cfd (a la hora de usar la nueva función del nuevo plugin)


Eduardo Vazquez
12:07
Seems like lower texas environments are available again
:marca_de_verificación_blanca:
1
:manos_levantadas::tono-de-piel-2:
1



Cristian Lorenzo
15:39
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/692
#692 test: prevent addition of shared-components styles into the DOM
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 17 abr. | Añadido por GitHub
15:40
Vamos a ver si así se evitan memory leaks
15:40
Y si es el caso, igual se puede proponer para pos-web/jest


Irene Foz Almagro
15:45
esto es algún error que sale cuando lanzas los tests en local?
15:45
igual que como esto del CSS que hay en PNR por ejemplo? https://github.com/Tvlport-SPC/pos-web-pnr/blob/3f844b388c08c3a017acffdc95638a4e23378ce7/jest.setup.js#L16


Cristian Lorenzo
15:51
Sí. Pero eso hace que no llegue a ocurrir. Lo otro oculta el mensaje
15:52
Pero el parseo del css + unwrap de sourcemaps nos lo comemos
15:52
Y ocupa la de Dios


Irene Foz Almagro
15:54
vale entiendo


Cristian Lorenzo
08:21
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/695. Estuve estudiando el consumo de memoria de los test y esta PR tiene algunos cambios que ayudarán:
Subida de plugin-reference-data a una nueva versión en que los data-sets del mock server se cargan lazy. Eso reduce el parseo de la copia de los dataset en cada beforeEach, reduciendo en el peor de los casos hasta 5 segundos en cada bloque de test.
Un selector devolvia un objeto vació nuevo en cada llamada, además de no estar memoizado. Esto generaba multiples re-renders falsos, lo que se notificaba en otros plugins como plugin-store-front,
El reporter para agrupar los mensajes de jest tiene un memory leak o uso ineficiente de memoria. Una vez eliminado, el crecimiento del consumo de memoria se ha estabilizado. Creo que siguen habiendo otros memory leaks, pero este al usarse en conjunción con runInBand era el de mayor impacto.
He subido la versión de jest y JSDOM.
Con todos estos cambios, he podido rehabilitar la ejecución en paralelo de los test a 4 workers, lo que pone la pipeline de sonarqube en 4 mins vs 10.
#695 refactor(plugin-sell): performance improvements
## Description
Performance improvements:
• Apply updates with critical performance improvements.
• Correct a bug with a selector returning a new empty object each re-render.
## 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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 21 abr. | Añadido por GitHub
:+1::tono-de-piel-2:
1
:manos_levantadas:
1



2 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
08:22
Este también está para revisar: https://github.com/Tvlport-SPC/pos-web/pull/864
#864 feat(app-kernel-session-manager): add workarea header
## Description
Add workarea header:
• It will be added when not set, and the hostconnectionid header is provided.
• Many plugins just consume the kernel or the legacy connection-selector plugin to add this header to the service calls. This will remove dependencies.
## 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.
Mostrar más
Comments
1
Tvlport-SPC/pos-web | 18 abr. | Añadido por GitHub
:+1::tono-de-piel-2:
1



Maryam Feizabad
09:13
@Cristian Lorenzo For the issue we faced in OB fees, I'm reviewing how we handle PRE_ET and POST_ET.
in PRE_ET, we can continue using getPrice, but in POST_ET, we need to switch to use the workbench.
Currently, in R&C "Issue Ticket", we fetch for initializeworkbench, which provides us with the offer and the totalPrice
image.png

image.png


Cristian Lorenzo
09:21
Ok, so we need to check if with those IDs the price check works.


Maryam Feizabad
10:27
teneis SignIn error en lower envs?

1 respuesta
hace 7 mesesVer hilo


Maryam Feizabad
12:06
@Irene Foz Almagro @Chema Martin
map the OB fees in getPrice:
image.png

image.png
:+1::tono-de-piel-2::+1:
2



12 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
15:42
Para revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/434
#434 refactor: performance improvements
## Description
Performance improvements:
• Change selectors that were not properly memoized, causing re-renders.
• Updated dependencies.
## 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
Comments
1
Tvlport-SPC/pos-web-payment | 22 abr. | Añadido por GitHub


Irene Foz Almagro
08:55
Os paso algunas PRs del bot:
build(deps-dev): bump the dependencies group across 1 directory with 24 updates by dependabot[bot] · Pull Request #55 · Tvlport-SPC/pos-web-telemetry
build(deps): bump the dependencies group across 1 directory with 49 updates by dependabot[bot] · Pull Request #312 · Tvlport-SPC/pos-web-plugin-reservation-details
build(deps): bump the dependencies group across 1 directory with 54 updates by dependabot[bot] · Pull Request #640 · Tvlport-SPC/pos-web-traveler-details
:marca_de_verificación_blanca:
1



Irene Foz Almagro
09:41
Tengo esta pequeña PR de mantenimiento, no he actualizado todo pero algunas dependencias y he elimnado varias cosas
Feature/maintenance work by Irene-FozAlmagro_tvlport · Pull Request #693 · Tvlport-SPC/pos-web-plugin-sell
:marca_de_verificación_blanca:
1



Israel Naranjo
10:38
podeis echarle un ojo y vamos comentando -> https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/9 (editado)
:ojos:
1
:+1:
1

10:38
:rezo:
10:43
tengo que borrar cosillas ...........


Irene Foz Almagro
10:55
puedes avisar cuando elimines las cosas que no hacen falta profa


Israel Naranjo
10:59
sip
:manos_levantadas::manos_levantadas::tono-de-piel-2:
2



Israel Naranjo
12:13
He quitado bastante @Irene Foz Almagro @Maryam
:+1::+1::tono-de-piel-2:
2



12 respuestas
Última respuesta hace 7 mesesVer hilo


Cristian Lorenzo
12:23
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/329
#329 refactor: performance improvements
## Description
Made improvements to get better performance and hermeticity:
• Remove side effects from testWith,
• Correctly memoize connectors that produced re-renders without state change.
• Remove plugin-connection-selector
## Is this change related to a Jira Task?
• Yes
• Story/task Link:
• Feature Link:
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-air-price | 23 abr. | Añadido por GitHub


Cristian Lorenzo
14:17
Para revisar https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1565
#1565 fix(plugin-reference-data): share lock on all instances
## Description
Share lock on all instances of dataset loaders:
• This will prevent concurrent calls when requesting concurrently both mapped and standard dataset hooks.
## 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.
• My changes generate no new warnings.
Mostrar más
Tvlport-SPC/pos-web-kernel-plugins | 23 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
16:08
Un voto más por aquí y mergeamos: https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1565
#1565 fix(plugin-reference-data): share lock on all instances
## Description
Share lock on all instances of dataset loaders:
• This will prevent concurrent calls when requesting concurrently both mapped and standard dataset hooks.
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 23 abr. | Añadido por GitHub
:+1::tono-de-piel-2:
1



Irene Foz Almagro
16:09
Tengo un problema con la PR de Chema, cuando lanzo los tests al principio me pedia unas dependencias y ahora estoy con este error y no lo entiendo del todo. Basciamente he agregado el testWith que me pedia y ahora me dice esto
image.png

image.png


Eduardo Vazquez
16:10
Esto es que hay algún error con el testWith del plugin-exchanges-api-connection


Irene Foz Almagro
16:10
Tienes 5 min qe creo que me puedes ayudar


Eduardo Vazquez
16:10
voyy


Maryam Feizabad
09:45
I'm personally hesitant to accept ownership of @pos-web/plugin-air-confirmation under Rumiko, as all the errors surfaced in the endTransact frontend would end up being directed to our team.
Given its functional scope, I believe it would make more sense go under the Shopping team's ownership

1 respuesta
hace 7 mesesVer hilo


Maryam Feizabad
12:42
FYI: Any testing Related to Air India development, we are Going to test under PCC: "6RH4"
:+1::tono-de-piel-2::+1:
3



Maryam Feizabad
13:01
any one here has knowledge the plugin-end-transact-gui? have u ever worked on this plugin?


3 respuestas
Última respuesta hace 7 mesesVer hilo


Maryam Feizabad
14:35
about Upsell the AirPriceFullPayload is already implemented and has the payload (editado)
apiV11.js
if (!isFullPayload) {
Tvlport-SPC/ts-communication-layer | Añadido por GitHub
:ojos:
1



Joan Perez camprubi
:eat: 15:29
Salgo a comer!


Irene Foz Almagro
09:36
Tengo una PR de mantenimiento para usar el biome en pos-web-telemetry build: migrate repository to use biome by Irene-FozAlmagro_tvlport · Pull Request #56 · Tvlport-SPC/pos-web-telemetry
:marca_de_verificación_blanca:
1
:manos_levantadas:
1


1 respuesta
hace 7 mesesVer hilo


Israel Naranjo
09:53
alguien tiene un ratillo , para deployar el proyecto en Rumiko ??? https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/pull/9
please help me :sonrisa_con_sudor:



20 respuestas
Última respuesta hace 7 mesesVer hilo


Joan Perez camprubi
:eat: 09:55
Review https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/644
Quitando FF
:+1::tono-de-piel-2:
1



Joan Perez camprubi
:eat: 11:01
Review https://github.com/Tvlport-SPC/pos-web-payment/pull/436
Añadí una guarda para que no pete cuando sea undefined, petaba en los tests del plugin-sell
:marca_de_verificación_blanca:
1




11 respuestas
Última respuesta hace 7 mesesVer hilo


Irene Foz Almagro
12:59
Quick PR build(packaging): update to next user-event dependency version by Irene-FozAlmagro_tvlport · Pull Request #123 · Tvlport-SPC/pos-web-workbench
:marca_de_verificación_blanca:
1



3 respuestas
Última respuesta hace 7 mesesVer hilo


Joan Perez camprubi
:eat: 13:14
Hola, queda alguien por aquí?

1 respuesta
hace 7 mesesVer hilo


Jose Martin
13:14
yup


Joan Perez camprubi
:eat: 13:15
Estas muy liado?
13:15
Sino te muestro una cosa aver si me puedes echar cable


Jose Martin
13:15
no, dime
13:15
te llamo


Joan Perez camprubi
:eat: 13:15
ok
13:15
webex pls
13:15
voy


Jose Martin
13:15
dame un segundo


Joan Perez camprubi
:eat: 15:13
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/682/


3 respuestas
Última respuesta hace 7 mesesVer hilo


Joan Perez camprubi
:eat: 16:07
Hasta el Lunes, buen finde!

1 respuesta
hace 6 mesesVer hilo


Irene Foz Almagro
08:59
Os paso algunas PRs de dependabot para que haya más de una persona que lo revise :cara_ligeramente_sonriente:
build(deps-dev): bump the dependencies group across 1 directory with 25 updates by dependabot[bot] · Pull Request #241 · Tvlport-SPC/pos-web-agency-tools
build(deps-dev): bump @pos-web/plugin-workbench from 3.21.3 to 3.21.4 in the dependencies group across 1 directory by dependabot[bot] · Pull Request #57 · Tvlport-SPC/pos-web-telemetry
build(deps-dev): bump the dependencies group across 1 directory with 7 updates by dependabot[bot] · Pull Request #81 · Tvlport-SPC/pos-web-plugin-notepad
build(deps-dev): bump the dependencies group across 1 directory with 2 updates by dependabot[bot] · Pull Request #124 · Tvlport-SPC/pos-web-workbench
build(deps): bump the dependencies group across 1 directory with 8 updates by dependabot[bot] · Pull Request #88 · Tvlport-SPC/fes-profiles-manager
build(deps): bump the dependencies group across 1 directory with 6 updates by dependabot[bot] · Pull Request #144 · Tvlport-SPC/fes-non-air-modify
build(deps): bump the dependencies group across 1 directory with 8 updates by dependabot[bot] · Pull Request #102 · Tvlport-SPC/fes-plugin-settings
:marca_de_verificación_blanca:
1


1 respuesta
hace 6 mesesVer hilo


Israel Naranjo
09:49
lo vemos en la daily


Maryam Feizabad
10:21
@Israel te refieres a esto?
https://github.com/Tvlport-SPC/fes-reservation-travelers-agency-manager/actions/runs/14383044634/job/40331433755?pr=4



25 respuestas
Última respuesta hace 6 mesesVer hilo


Cristian Lorenzo
11:44
Para revisar, si podéis, https://github.com/Tvlport-SPC/pos-web-kernel-plugins/pull/1568
#1568 test(plugin-reference-data): export mocks as js with JSDOC typings
## Description
Export mocks as js with JSDOC typings:
• Makes easier to consume them by dependants.
## 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.
• My changes generate no new warnings.
Mostrar más
Comments
1
Tvlport-SPC/pos-web-kernel-plugins | 29 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1


1 respuesta
hace 6 mesesVer hilo


Cristian Lorenzo
11:44
Y así mergeo


Maryam Feizabad
13:43
Les comparto las stories, Mirad y decidme si queréis que hagamos una llamada entre nosotros:
https://travelport.atlassian.net/browse/EV-44348
EV-44348 TP+ - SPC Allow customers to Price booked itinerary with upsells in…
Status: Implementing
Type: Epic
Assignee: Florencia Couto
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud
:ojos:
1




4 respuestas
Última respuesta hace 6 mesesVer hilo


Irene Foz Almagro
14:35
Os paso una PR que hice ayer antes del apagón, es de mantenimiento build: migrate repository to use biome by Irene-FozAlmagro_tvlport · Pull Request #125 · Tvlport-SPC/pos-web-workbench


4 respuestas
Última respuesta hace 6 mesesVer hilo


Joan Perez camprubi
:eat: 14:46
Hello


Joan Perez camprubi
:eat: 14:46
Alguien se anima a ayudarme?



93 respuestas
Última respuesta hace 6 mesesVer hilo


Irene Foz Almagro
16:08
@Maryam hacemos la meeting para poner story points en la feautre que hemos estado mirando esta mañana?


Maryam Feizabad
16:15
Están instalando Internet abajo, os aviso
:+1::tono-de-piel-2:
1



Israel Naranjo
16:28
lo dejamos para mañana , para no andar con prisa?
:+1::+1::tono-de-piel-2:
2



Irene Foz Almagro
16:29
Si yo creo que mañana con tranquilidad lo hacemos, estoy revisando las stories y tenemos todas con descripciones. (Gracias @Maryam :cara_ligeramente_sonriente: )


Maryam Feizabad
16:30
ya estoy


Irene Foz Almagro
16:31
si queres hacemos de 30 min para ver si hay preguntas o algo


Israel Naranjo
16:31
You have to bring us, a gift from Japon @Maryam
:sonrisa_con_sudor:
1



Maryam Feizabad
16:31
hablas muy bien Ingles, eee


Israel Naranjo
16:32
Norrrrr


Eduardo Vazquez
16:32
日本で楽しんでください!!
:alegría:
3

16:33
(que te lo pases muy bien hehe)


Irene Foz Almagro
16:34
os vamos a invitar a la meeting para hablar este rato de la feature


Joan Perez camprubi
:eat: 16:36
Otsukaresama @Maryam
:alegría:
2

16:38
Yuimaru Rumiko!


Cristian Lorenzo
10:07
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/314
#314 refactor: remove plugin-connection-selector
## Description
Remove plugin-connection-selector:
• All usages are done via kernels injecting headers.
## Motivation and Context
Reduce complexity
## 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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 30 abr. | Añadido por GitHub
:marca_de_verificación_blanca:
1



8 respuestas
Última respuesta hace 6 mesesVer hilo


Joan Perez camprubi
:eat: 11:26
Alguien esta usando rumiko?




65 respuestas
Última respuesta hace 6 mesesVer hilo


Jose Martin
11:30
ha respondido a un hilo de la conversación:
Alguien esta usando rumiko?
lo haces tú Joan?
Ver respuestas más recientes


Jose Martin
12:39
Pequeña PR para evitar multiples llamadas al getPrice: https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/335
#335 feat(plugin-pricing-api-connection): Avoid several requests when card number under six digits
## Description
This PR tries to avoid calling several times the getPrice endpoint (which takes some time) when adding card numbers under 6 digits which is also the amount of digits sent in the request.
## Is this change related to a Jira Task?
• Yes
• Story/task Link: https://travelport.atlassian.net/browse/EV-44343
• Feature Link: https://travelport.atlassian.net/browse/EV-35890
• 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
Tvlport-SPC/pos-web-plugin-air-price | 30 abr. | Añadido por GitHub
:+1::tono-de-piel-2:
1



9 respuestas
Última respuesta hace 6 mesesVer hilo


Irene Foz Almagro
12:44
@Israel he hecho una primera PR en tu fes, he puesto detalles en la template de la PR (creo que me fallaba el coverage pero después de comer lo miro) build: update yarn and apply necessary changes for Biome formatting by Irene-FozAlmagro_tvlport · Pull Request #11 · Tvlport-SPC/fes-reservation-travelers-agency-manager
:manos_levantadas:
1



Cristian Lorenzo
13:21
Para revisar https://github.com/Tvlport-SPC/pos-web-agency-tools/pull/242
#242 refactor: remove plugin-connection-selector
## Description
Remove plugin-connection-selector
## Motivation and Context
Reduce complexity of plugins' graph.
## 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
Comments
1
Tvlport-SPC/pos-web-agency-tools | 30 abr. | Añadido por GitHub
:+1:
1



Jose Martin
13:39
Mirad esto cuando podáis please



Jose Martin
Pequeña PR para evitar multiples llamadas al getPrice: https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/335
Hilo en rumiko_developers | 30 abr. | Ver el mensaje
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 14:21
Alguien se acuerda como skipear escencarios de gherkin?


Cristian Lorenzo
14:22
@Skip
14:22
Pero te fallará el CI


Joan Perez camprubi
:eat: 14:22
porque?


Cristian Lorenzo
14:22
Porque está prohibido por una regla hacer skip


Joan Perez camprubi
:eat: 14:22
pero en test de jest si?
14:23
Bueno sino los comento y ya esta
14:24
en test de jest si se puede pero en gherkin scenarios no?


Cristian Lorenzo
14:24
Si



Joan Perez camprubi
:eat: 09:43
Han puesto una nueva company call mañana por la mañana
09:43
Es lo mismo que la de hoy?


Adrian Lopez
:prohibido_el_paso: 09:43
tiene pinta que si
09:44
"Greg will host this call for Asia Pacific colleagues live from the UK, which is why it’s scheduled at a different time than usual."
:+1:
1



Joan Perez camprubi
:eat: 09:44
a ostia
09:44
que no lo he leido
:muriéndose_de_risa:
1



Cristian Lorenzo
10:22
https://support.travelport.com/webhelp/JSONAPIs/Airv11/Content/Air11/Book/APIRef_ReservationRetrieve.htm


Cristian Lorenzo
14:37
Necesito calcular la lista de adyacencia de la transacción a tiempo arbitrario. Esa utilidad la tienes ya @Manuel Moraga? (editado)
14:38
Es para retornarla compatificada, si no o bien la programo, o bien de momento retorno la transacción reconciliada sin compactificar (editado)


Manuel Moraga
14:44
aún no la tengo


Pedro Morata Vila
15:55
pues a la mierda, vaya perdda de tiempo


Jose Martin
15:56
esta película me suena haberla visto ya antes
15:56
es un remake (editado)


Adrian Lopez
:prohibido_el_paso: 15:56
Esta Llamada Podria Haber Sido Un Email 2


Joan Perez camprubi
:eat: 15:58
xD


Israel Naranjo
15:58
si par ver al ceporro prepotente
15:59
cerdo (2 MB)
cerdo


Shared with the Giphy shortcut


Joan Perez camprubi
:eat: 16:00
Queda claro que su plan numero 1 es que la gente se vaya por su propio pie


Israel Naranjo
16:01
se ahorran un pastizal


Cristian Lorenzo
16:08
Para ir revisando https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/256. No sé como irá el sonarqube, dio timeout
#256 Feat/reconciliation with buckets
Description
First batch on the reconciler. This package will be renamed, therefore a following this merge a new one will follow.
Motivation and Context
Part of https://travelport.atlassian.net/browse/EV-42903
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-plugin-reservation-details | 13 ene. | Añadido por GitHub


Pedro Morata Vila
16:15
"Necesito calcular la lista de adyacencia de la transacción a tiempo arbitrario"
Mande????


Cristian Lorenzo
16:17
Para transacción después de n pasos, obtener como estaría la parte de relations. Las relations son una lista de adyacencia:
{ [entityID]: [lista de ids de hijos] } (editado)


Israel Naranjo
16:17
:cara_con_ojos_en_espiral:


Cristian Lorenzo
16:18
Para compactar, es decir, no retornar operaciones que no interesan, necesito eliminar todos los descendientes
:+1:
1

16:19
Es decir, si borro el Pasajero 3, puedo tambien eliminar sus Documentos y todas las operaciones que los actualicen


Adrian Lopez
:prohibido_el_paso: 16:41
image.png

image.png
16:41
adivinad porque petaba el test (editado)
16:42
(al parecer petaba porque la expiration date es de este mes, y no lo acepta por alguna razon, muy random)
:+1::tono-de-piel-5:
1



Adrian Lopez
:prohibido_el_paso: 17:44
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/561
Una de las 5 PRs, solo habia que quitar la string (en este caso) y lo del test de arriba.
#561 [DRAFT] refactor: removal of ft-review-and-confirm feature flag
Description
This PR removes the FT-REVIEW-AND-CONFIRM feature flag across all SPC-related repositories.
This is done across pos-web-traveler-details, pos-web-plugin-sell, pos-web-modify, pos-web-plugin-traveler-information and pos-web-ticket.
In this repo, the single reference to the flag in a constant has been deleted; it doesn't exist elsewhere in the repo.
Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-43077
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… Mostrar más
Comments
1
Tvlport-SPC/pos-web-traveler-details | 14 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
3



Pedro Morata Vila
09:38
very bright future
09:38
aha


Adrian Lopez
:prohibido_el_paso: 11:36
Para hacer reviews:
* Sell (Arya): https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/569
* Modify (Decebal): https://github.com/Tvlport-SPC/pos-web-modify/pull/208
* TravelerInformation (Dracula): https://github.com/Tvlport-SPC/pos-web-plugin-traveler-information/pull/260
* Ticket (Picasso): https://github.com/Tvlport-SPC/pos-web-ticket/pull/393
:marca_de_verificación_blanca:
1



David Vazquez
12:29
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/255
#255 feat(retain-review-and-confirm-state): add method to create a checkpoint from form state
Description
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 13 ene. | Añadido por GitHub


David Vazquez
14:44
otra más para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/570
#570 feat(plugin-sell): add method to create a checkpoint from formstate
Description
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-plugin-sell | 16 ene. | Añadido por GitHub


Jose Martin
15:14
Para ir mirando: https://github.com/Tvlport-SPC/pos-web-seat-maps/pull/393
#393 fix(seat-maps-gui): avoid readonly traveler fields clearance after filling seatmaps form
Description
This PR fixes a defect when filling a editable traveler information in the seat maps form (Editing a traveler would clear any other readonly field in the form).
Is this change related to a Jira Task?
• Yes
• Story Link:
• Feature Link: https://travelport.atlassian.net/browse/EV-39823
• 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-seat-maps | 15 ene. | Añadido por GitHub


Cristian Lorenzo
15:40
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/259
#259 refactor(reservation-details-sync-engine): move reconciler to the new…
Description
Rove reconciler to the new package reservation-details-sync-engine. We keep it on the old one
to prevent conflicts while working on parallel on the feature.
Motivation and Context
We will be removing the old package as it has non-descriptive name.
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
Mostrar más
Tvlport-SPC/pos-web-plugin-reservation-details | 16 ene. | Añadido por GitHub


Joan Perez camprubi
:eat: 17:04
Nois


Joan Perez camprubi
:eat: 17:05
Viendo que le queda poco al spike de Isra, como veis lo del flexi? @aquí



5 respuestas
Última respuesta hace 10 mesesVer hilo


Jose Martin
17:15
Mi tarea está ya (EV-39823 ) sólo pendiente que me la acepte el reporter que no me responde.
EV-39823 [Seats][PreET][PaxDetails] If entering details for 2nd pax, 1st pax…
Status: In Acceptance
Type: Defect
Assignee: Jose Martin
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud


4 respuestas
Última respuesta hace 10 mesesVer hilo


Cristian Lorenzo
11:48
Para revisar: https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/261
#261 feat(reservation-details-sync-engine): add relations management for p…
Description
add relations management for persons
• It required fixing issues with Heap going out of boundaries and inverse ordering.
• Added feature file for next step
Motivation and Context
Advance with feature
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
Mostrar más
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 20 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
2

11:48
Para cada entidad, añadid el feature file desde un principio porque es algo complicado
11:48
packages/reservation-details-sync-engine/src/methods/reconcileTransactions/__test__/reconcile-transactions__documents-subgraph.feature


Joan Perez camprubi
:eat: 11:53
Creamos un test (feature file, steps) para cada entidad no?
11:56
En esta PR, creaste el feature file de documents, pero no lo estás usando en ningun test no?


Cristian Lorenzo
12:11
Sí, cread para cada entidad mínimo uno
12:12
No lo estoy usando, luego lo implemento. He metido ya la PR porque he cambiado algunos steps para que sirvan a transacciones form/external y así no hay conflictos
:+1:
1



David Vazquez
15:58
pequeña PR para poder añadir un travel document con el mocks-factory https://github.com/Tvlport-SPC/pos-web-pnr/pull/1652
#1652 feat(pnr-mocks-factory): Add data argument to createPersonManualSpeci…
…alServiceRequests for additional flexibility
Description
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
Tvlport-SPC/pos-web-pnr | 20 ene. | Añadido por GitHub
15:59
esta PR también se puede revisar y mergear, es la que mueve la funcionalidad de crear el checkpoint del formulario al plugin-sell https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/570
#570 feat(plugin-sell): add method to create a checkpoint from formstate
Description
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 16 ene. | Añadido por GitHub


Cristian Lorenzo
17:38
Para revisar https://github.com/Tvlport-SPC/fes-non-air-modify/pull/119
#119 fix: import cjs with multiple imports
Description
import cjs with multiple imports
Motivation and Context
Incompatibility between module systems
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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Comments
1
Tvlport-SPC/fes-non-air-modify | 20 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1

17:41
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/262
#262 feat(reservation-details-sync-engine): implement Document sync with p…
Description
implement Document sync with prior given feature file
Motivation and Context
Part of https://travelport.atlassian.net/browse/EV-43075
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 20 ene. | Añadido por GitHub


Israel Naranjo
09:42
lo podeis revisar porfa https://travelport.atlassian.net/wiki/spaces/EV/pages/51430160/Spike+Validation+of+Minimum+Connection+Time+MCT+in+Multi-Segment+Reservations

SPIKE: Validation of Minimum Connection Time (MCT) in Multi-Segment Reservations
Spike: Validation of Minimum Connection Time (MCT) in Multi-Segment Reservations
Summary: This spike is all about developing an automated system to spot errors in Minimum Connec...
Owned by: isnave | 1 person viewed
More actions...
Añadido por Confluence Cloud


David Vazquez
11:38
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/263
#263 feat(reservation-details-sync-engine): add method to create a checkpoint from a pnr
Description
Implement method to create a checkpoint from a PNR data using the entities interface
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 20 ene. | Añadido por GitHub


David Vazquez
11:51
otra PR más para revisar resolviendo los comentarios https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/570
#570 feat(plugin-sell): add method to create a checkpoint from formstate
Description
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 16 ene. | Añadido por GitHub


Manuel Moraga
12:06
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/264
#264 feat(reservation-details-sync-engine): add method to get a partial pn…
Create a method to get a partial PNR from a transaction
Description
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-plugin-reservation-details | 21 ene. | Añadido por GitHub

1 respuesta
hace 10 mesesVer hilo


Manuel Moraga
12:06
en un rato salgo al médico para un radiografía que tengo a las 1


David Vazquez
12:23
ya he hecho los cambios en las PR's @Cristian Lorenzo


2 respuestas
Última respuesta hace 10 mesesVer hilo


Cristian Lorenzo
12:36
Para revisar https://github.com/Tvlport-SPC/pos-web-payment/pull/397
#397 fix(payment-gui): add defensive code on received partial receipts fro…
Description
Add defensive code on received partial receipts from a workbench
• Part of fix of https://travelport.atlassian.net/browse/EV-43231
Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43231
• 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-payment | 21 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1



David Vazquez
16:27
podéis echar un ojo a esta PR https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/263? merci
#263 feat(reservation-details-sync-engine): add method to create a checkpoint from a pnr
Description
Implement method to create a checkpoint from a PNR data using the entities interface
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 20 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
2



Jose Martin
17:17
Esto se puede ir revisando, a la espera de subirlo a rumiko y ver si funciona bien: https://github.com/Tvlport-SPC/fes-traveler-information/pull/154
#154 fix(fes-traveler-information): update logic to compare email addresses and type
Description
Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43078
• 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
Tvlport-SPC/fes-traveler-information | 21 ene. | Añadido por GitHub


Cristian Lorenzo
09:02
Estuve pensando ayer, y hablando con el chatgpt, y podemos hacer eso del arbol de decisión. En el caso que nos ocupa, como hay muchos datos wildcard en las columnas, con wildcards implicitos. Tendríamos la estructura
interface Node {
mct: null | number;
kind: 'Airline' | 'OperatedBy' | ...
value: string | boolean | ...
children: Node[];
}
Por cada regla de la tabla, insertamos el camino en el arbol. Cuando queremos verificar las reglas, exploramos el árbol y para cada nodo que visitemos que tenga el valor mct diferente de null, añadimos a un acumulador el mct y la profundidad del camino. Cuando acabamos, si hay algun mct, retornamos el mtc de mayor profundidad (más específico). Si no, devolvemos el standard time del aeropuerto. (editado)


Israel Naranjo
09:05
interface MCTNode {
connectionType: string;
carrierCode?: string;
cabinClass?: string;
operatingCarrier?: string;
flightRange?: FlightRange;
children?: MCTNode[];<----
mct?: string;
isSuppressed?: boolean;
}
nos debio decir algo parecido
09:05
estoy viendo el video


Cristian Lorenzo
09:06
La mayoria de datos de un nodo serán undefined, es mejor usar la dupla kind, value
09:06
Si no has de mirar que valor es no nullish
09:09
Si el set es muy muy grande, se peude hacer un refinamiento usando un bloom filter o similar para lo que decía de falsos positivos. Se puede hacer una llave sintética de texto con el path y meterlo. El problema son los dos valores de rangos (fechas y número de vuelo)
09:12
Que o bien se exploran primero (Carrier -> Date -> Flight number -> bloom filer --> POSITIVE -> Exploramos el resto del árbol), o bien se ponen como presente o ausente (tipo key Carrier:{DATE_RANGE?: true : false}::...) y no se exploran pero cualquier rango obliga a mirar todo el árbol.
09:13
Yo miraría hacer primero el árbol y si vemos que es lento o se va de madre, ya el refinamiento del pre-filtrado
09:14
En todo caso, para este sprint hay que hacer ya lo del mensaje de error
09:14
Antes que ponerse con la mejora
09:15
Pues es lo que nos comprometimos.


Israel Naranjo
09:15
pero aun falta que nos digan algo de ux (editado)
09:15
no?


Cristian Lorenzo
09:18
No. El error que hay ahora se ha de ir y poner un mensaje que explique que está pasando
09:19
Cuando esto falla, mete un pete irrecuperable y te manda a la pantalla de urrecoverable error
09:20
En esa pantalla en lugar de mostrar el mensaje chungo de JSON API, se ha de poner uno que entienda el agente como The selected offer has MCT (minimum connection time) conflicts with flights in the reservation


Israel Naranjo
09:21
vale vale.. pense que lo hariamos del tiron


Cristian Lorenzo
09:21
En lugar del 'DATE SEQUENCE IN ITINERARY NEEDS VERIFICATION'
09:21
No, del tirón no se puede hacer. Se entrega a partes
09:21
Esa primero


Israel Naranjo
09:21
dale me pongo con ello


Cristian Lorenzo
09:22
Guay. Con eso se tiene lo minimo y ya hacemos bien la feature con los checks


Israel Naranjo
10:44
abro un bug acerca de esto
image.png

image.png

4 respuestas
Última respuesta hace 10 mesesVer hilo


Cristian Lorenzo
10:57
Sí, y sobre el control de teclado. Algo ha cambiado con un update de atlas


Israel Naranjo
11:00
después peta
image.png

image.png



4 respuestas
Última respuesta hace 10 mesesVer hilo


Cristian Lorenzo
11:07
Para revisar https://github.com/Tvlport-SPC/pos-web-custom-keys/pull/391
#391 fix: solve boost issue
Description
Update dependencies to potentially solve boost issue
Motivation and Context
Maintenance
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
Comments
1
Tvlport-SPC/pos-web-custom-keys | 22 ene. | Añadido por GitHub


Jose Martin
11:08
que es el boost issue este?

1 respuesta
hace 10 mesesVer hilo


Jose Martin
11:15
Esto se puede mirar ya: https://github.com/Tvlport-SPC/fes-traveler-information/pull/154
#154 fix(fes-traveler-information): update logic to compare email addresses and type
Description
Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43078
• 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/fes-traveler-information | 21 ene. | Añadido por GitHub


Israel Naranjo
11:29
@Chema Martin https://travelport.atlassian.net/browse/EV-43335

EV-43335 Error when adding when modifying the reservation, combos do not loa…
Status: To Do
Type: Defect
Assignee: Unassigned
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud


Jose Martin
11:32
Por lo que se ve es un bug que están arreglando los de atlas: https://travelport.atlassian.net/browse/ATLAS-221
ATLAS-221 Select - disabled value
Status: To Do
Type: Defect
Assignee: Edgar
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud


Israel Naranjo
11:33
la reserva se va a tomar por culo


Cristian Lorenzo
13:54
Yo me reserve esta hora de comer para temas personales hace unos días y no podré ir a lo del traspaso. Que vaya gente nuestra de ingeniería (editado)
:+1:
3

13:54
Tenéis invite? Si no le pediis a Damares


Pedro Morata Vila
13:54
yo tengo el invite, pero no soy el mas adecuado


Cristian Lorenzo
13:55
Bueno, es como lo de ayer pero otro dominio
13:55
Mas o menos a todo el mundo le va bien salvo que tenga algo urgente


Jose Martin
13:55
si, estamos invitados. Pediremos que se grabe también si no se acuerdan


Cristian Lorenzo
13:55
Guay
13:55
Gracias
:+1:
1



Jose Martin
15:12
vaya rollo esto del ppm
15:14
le doy un mes de vida al flexi, como máximo. Seguro que se lo cargan también.


David Vazquez
15:25
puede ser


Cristian Lorenzo
15:29
Ya estoy aquí. Que pasó?


Jose Martin
15:30
Bueno, el tema del nuevo PPM. Hay que reportar en que dedicamos el tiempo cada día


Cristian Lorenzo
15:30
Pero no se hacia ya? Hay que poner más específico?


Jose Martin
15:31
si, más específico
15:31
image.png

image.png


Cristian Lorenzo
15:31
Que pereza
15:32
También pueden poner una casilla sobre las horas que gastaré cada semana en rellenar esas casillas
:+1:
1
:muriéndose_de_risa:
2



Jose Martin
15:32
jajaja


Cristian Lorenzo
15:36
Podemos hacer comentarios al documento?


Jose Martin
15:36
diría que no


Adrian Lopez
:prohibido_el_paso: 15:36
creo que se puede poner cosas en el slido pero
15:37
en el slido o en #accessui-accessui01-ppm-2025


Cristian Lorenzo
15:37
La asignación está mal salvo que guion signifique lo mismo que la linea de arriba
15:37
Profile & Preferencing lo llevamos nosotros
15:38
En Agency Automation también estamos


David Vazquez
15:38
yo creo que el guión se refiere a la linea de arriba


Jose Martin
15:39
creo que se han dejado cosas de nuestro ownership, si
15:41
y quién va a validar esto, como se va a saber si se ha hecho bien o no. Nuria? (editado)


Pedro Morata Vila
15:44
nadie ha preguntado como se imputa el flexy, no?

1 respuesta
hace 10 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 15:44
creo que no, no han comentado nada por ahora


Cristian Lorenzo
15:45
Vaya bajón de cultura de empresa
:soñoliento:
1

15:45
Dedicar tiempo a estas cosas


Adrian Lopez
:prohibido_el_paso: 15:46
por lo que veo nos piden que como muy especifico pongamos 1h, nada de poner minutos


Pedro Morata Vila
15:47
menos mal que he recordado comer un poco a la 1... madre mia. ahora tenemos el townhall


Adrian Lopez
:prohibido_el_paso: 15:47
o sea es igualmente una pesadez eso si


Adrian Lopez
:prohibido_el_paso: 15:48
ha respondido a un hilo de la conversación:
nadie ha preguntado como se imputa el flexy, no?
puede o no puede ser que alguien haya preguntado esto en slido ahora


Cristian Lorenzo
15:49
Aun está la renión?


David Vazquez
15:49
yo no tengo el townhall


4 respuestas
Última respuesta hace 10 mesesVer hilo


Adrian Lopez
:prohibido_el_paso: 15:50
sep, aun esta la reunion y lo del townhall luego paso enlace si eso


2 respuestas
Última respuesta hace 10 mesesVer hilo


Cristian Lorenzo
15:50
Como la rechazé no la tengo. Bueno, ya la veré
15:52
En video


Joan Perez camprubi
:eat: 15:53
Pero no se supone que los story points para cada tarea ya servian para eso?
15:53
Esque además es muy relativo
15:53
Porque puedo estar haciendo multiples cosas a la vez


Cristian Lorenzo
15:54
No, no sirven. Se puede planificar por horas de trabajo, pero es más dificil atinar bien


Joan Perez camprubi
:eat: 15:54
Por ejemplo, las tareas de deployment, que tengo que poner el rato que he gastado mirando la pipeline, arreglandola, y relanzandola?
15:54
O el tiempo que ha durado en hacer todo el ci/cd?
15:54
Es muy dificil eso, además subjetivo a cada uno
15:54
Lo veo un absurdo
15:55
Lo que tendrían que hacer es monitorizar, y ver las entregas, problemas de cada equipo tarea, para ver en donde se han dedicado las horas, si son efectivas, si han habido agentes externos que han afectado. No creo que sea una buena manera que cada uno lo reporte como le parezca
15:56
Eso lo deberían de controlar ellos..


Cristian Lorenzo
15:56
Ya, es un horror.


Joan Perez camprubi
:eat: 15:56
Como mucho diferenciar, horas efectivas de desarrollo, planificación, creación de diseños tecnicos
15:58
Pero también es una manera de ejercer presión a aver si tardo más en programar, o en hacer un diseño tecnico, y tener que estar pensando en eso, o no tardar más de lo que es esperado
15:58
Esque para eso ya está jira, y el rato que se tira cada ticket en cada columna, no?
15:58
Y lo que son las meetings y ceremonias, eso ya se puede calcular con los calendarios de outlook


Pedro Morata Vila
16:00
https://tvpt.webex.com/webappng/sites/tvpt/meeting/webinar/706e9ba73ad84698b45f55c53[…]imestamp=1737557982253&MTID=mcb4eac4fb55a0905f01bee216b2392a4
Cisco Webex SiteCisco Webex Site
Meet virtually with Cisco Webex. Anytime, anywhere, on any device.
Simple, modern video meetings for everyone on the world's most popular and trusted collaboration platform. (15 kB)
https://tvpt.webex.com/webappng/sites/tvpt/meeting/webinar/706e9ba73ad84698b45f55c533799a4e?ticket=4832534b000000078e17db45a519c12297c08611a66ba6a075de0510585f958d423651c5b8c77245×tamp=1737557982253&MTID=mcb4eac4fb55a0905f01bee216b2392a4

:+1:
1



Joan Perez camprubi
:eat: 16:00
y esto?


Adrian Lopez
:prohibido_el_paso: 16:00
:boutta_lose_it:
image.png

image.png


Pedro Morata Vila
16:01
el link de la reunion


Adrian Lopez
:prohibido_el_paso: 16:01
tienen tan poco dinero que se ahorran la ultima l de townhall
:alegría:
2



Joan Perez camprubi
:eat: 16:01
ah vale


Pedro Morata Vila
16:01
madre de dios.... he tenido que poner mi email 3 veces.


Joan Perez camprubi
:eat: 16:01
Se puede poner feedback, a esto del PPM?
:alegría:
1

16:05
5 minutos, esperando que empieze el webcast
16:05
Eso habrá que reportarlo también
:muriéndose_de_risa:
1

16:05
jaja
16:08
Hometown no sería Manchester city?


Jose Martin
16:13
hombre, creo que se entiende así también :sonrisa:


Joan Perez camprubi
:eat: 16:14
jajaja
16:14
o City of Manchester
16:18
Manchester City (2 MB)
Manchester City


Posted using /giphy | GIF by BT Sport


Israel Naranjo
16:46
cada vez que alguien le dan la gestion de cosas , se crecen a lo grande , como el saFe


Cristian Lorenzo
16:58
Las horas extra donde las imputo, eso tendría que ir al slido jjj
16:58
O huelga de celo, ni un minuto de más
:+1:
2



Israel Naranjo
17:01
ponlo
17:01
lo apollare
:pollo:
1
:pollito_saliendo_del_cascarón:
1

17:01
eso lo hacia en proyectos pequeños o que desconfian de lo que haces
17:02
pero estamos llenado hacia atras
17:07
gallo (15 kB)
gallo


Shared with the Giphy shortcut


Cristian Lorenzo
17:15
Si, es un paso atrás en cultura de empresa


Israel Naranjo
17:16
yo lo he puesto , que sea lo q tenga que ser. votadlo cabrones


Jose Martin
17:16
no quiero ser cenizo, pero al flexi le veo los días contados


Israel Naranjo
17:17
no sera mobbing, par kitarse gente , que terminemos hasta la polla?
17:17
:cabeza_explotando:


Joan Perez camprubi
:eat: 17:19
donde se vota?


Joan Perez camprubi
:eat: 17:20
pasa el link @Israel

2 respuestas
Última respuesta hace 10 mesesVer hilo


Jose Martin
17:41
Esta PR arregla lo de los select desactivados no enseñando el value: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/565
#565 fix(plugin-traveler-information-gui): fix disabled selects not showing value
Description
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 | 22 ene. | Añadido por GitHub


David Vazquez
09:37
PR para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/265
#265 feat(reservation-details-sync-engine): create method to transform a transaction to form state
Description
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 22 ene. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 14:43
Para revisar (es la primera parte de la tarea, permitiendo el nuevo tipo de header y de respuesta): https://github.com/Tvlport-SPC/fes-plugin-settings/pull/80 (editado)
#80 feat: add new schema interface and versioning
Description
This JIRA adds a Middleware that overwrites the default res.json() in this /api service, allowing us to intercept the resulting response and morph it into a specific interface.
Motivation and Context
JIRA: TBA
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
Comments
1
Tvlport-SPC/fes-plugin-settings | 22 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
15:17
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/573
#573 feat(workflow-manager-core): add simple scheduler
Description
Add simple scheduler to the workflow library
Motivation and Context
Required to have flows based on atomic tasks.:
• Added core entities Task, ExecutableTask, and Flow.
• Flows can create a execution plan based on a simple scheduler that computes the topological order of the dependency graph. It does not consider parallelism.
• Added tests for negative plan cases for invalid types, inputs/output pairs, circular dependencies, and multiple producers.
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)
Ty… Mostrar más
Tvlport-SPC/pos-web-plugin-sell | 23 ene. | Añadido por GitHub


Israel Naranjo
16:37
pa revisar https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/566
:marca_de_verificación_blanca:
1




4 respuestas
Última respuesta hace 10 mesesVer hilo


David Vazquez
18:21
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/266
#266 feat(reservation-details-sync-engine): export method transactionToFor…
…mState
Description
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-plugin-reservation-details | 23 ene. | Añadido por GitHub


David Vazquez
19:45
si le podéis volver a dar approve os lo agradecería, tenía que arreglar una cosa


Cristian Lorenzo
09:51
Para revisar https://github.com/Tvlport-SPC/fes-plugin-settings/pull/82
#82 fix: solve issues with transpiling vs no-transpiling import of logger
Description
solve issues with transpiling vs no-transpiling import of logger
Motivation and Context
Unable to start service
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/fes-plugin-settings | 27 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
10:15
https://github.com/Tvlport-SPC/pos-web-devkit-integration
Tvlport-SPC/pos-web-devkit-integration
Plugins and Packages to integrate Devkit tools into POS Web
Language
TypeScript
Last updated
4 months ago
Añadido por GitHub


Cristian Lorenzo
13:09
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/575
#575 feat(workflow-manager-core): add InMemoryFlowRunner
Description
Add InMemoryFlowRunner:
• Missing part of reversing on failure
Motivation and Context
Partial deliverable of this class implementation
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-plugin-sell | 27 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1

13:10
Voy a comer mientras pasa pipelines


Cristian Lorenzo
14:29
Ya arreglé lo que dijo copilot
14:29
Se puede revisar


David Vazquez
16:31
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/553


David Vazquez
18:13
ya arreglé el comentario @Cristian Lorenzo (editado)
:marca_de_verificación_blanca:
1



Israel Naranjo
18:16
mañana quien va a la oficina?


David Vazquez
18:16
yo iré seguramente


Israel Naranjo
18:16
rezare ,


David Vazquez
18:17
rezarás para qué?


Israel Naranjo
18:17
entre lo s que esten malos y demas , no me apetece estar solo


Adrian Lopez
:prohibido_el_paso: 18:17
yo voy


David Vazquez
18:18
Cristian dijo que iba también no?


Israel Naranjo
18:18
:brazo_mecánico:


Joan Perez camprubi
:eat: 18:19
Yo iré también


David Vazquez
18:20
pues ya está Israel, ya no tienes excusa


Israel Naranjo
18:21
perraca


Cristian Lorenzo
19:56
Si, yo voy


Cristian Lorenzo
09:03
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/576
#576 test(workflow-manager-core): add after scenario
Description
Added after scenario and rollback functionality
Motivation and Context
Required for task based flows
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 27 ene. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 16:04
Para revisar: https://github.com/Tvlport-SPC/fes-plugin-settings/pull/83 (añade un test y un tipo que puede ocurrir en un parametro al devolver el resultado nuevo)
#83 test: add interface missing typing and test regarding schema fetching
Description
This PR adds a missing alternative on the typing of SPCSettingsResponse and adds another test that was initially missing on the previous commit.
Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-43165
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/fes-plugin-settings | 28 ene. | Añadido por GitHub


Cristian Lorenzo
16:14
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/577
#577 Feat/add support for provided values
Description
Add support for provided values, which are typed flow inputs not provided by any task of the flow.
Motivation and Context
Needed for the reference implementation
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-plugin-sell | 28 ene. | Añadido por GitHub


David Vazquez
16:28
para revisar https://github.com/Tvlport-SPC/pos-web-fare-shop/pull/976
#976 fix(plugin-fare-shop-search): allow search in pnr bookings that only contains ATX, TUR or SUR passive segments
Description
Make an exception on the error message showing decision tree to allow bookings to be started from PNRs that only contain passive segments such as ATX, TUR or SUR segments:
Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43237
• 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
Tvlport-SPC/pos-web-fare-shop | 28 ene. | Añadido por GitHub


Manuel Moraga
11:40
estoy en la vpn LHR y cuando trato de acceder a https://sp-pp.travelport.com/ me tira:
image.png

image.png
11:40
os pasa o tenéis idea que pueda ser?


Cristian Lorenzo
12:01
Para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/579
#579 Docs/update task runner readme with mvp
Description
Update task runner readme with MVP
Motivation and Context
Required for first release/contact
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 29 ene. | Añadido por GitHub


Pedro Morata Vila
12:19
te dice lo mismo con atl?


Manuel Moraga
12:29
si


Joan Perez camprubi
:eat: 14:41
Estaba reproduciendo con Manu el defect que esta llevando ya que no le va pp
14:41
Hemos visto que las contraseñas se ven crudas en el payload


Cristian Lorenzo
14:42
En que payload? El que se manda a la API?


Joan Perez camprubi
:eat: 14:42
Al ser https supongo que no hay problema
14:43
El que el navegador manda al servicio
14:43
No te lo pueden esnifar eso?


Adrian Lopez
:prohibido_el_paso: 16:03
Para revisar (primera PR para retirar el modifyTravelerSectionsEditability del antiguo RnC y asi quitar un test que peta el sell al hacer el fix): https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/580
#580 refactor: remove modifyTravelerSectionsEditability action consumption
Description
This PR intends to remove modifyTravelerSectionsEditability's action usage from plugin-sell, in order to refactor and clean up old code. This action comes from the legacy Review and Confirm.
Motivation and Context
Refactor done in order to clean up for a future PR (#578) that fixes a critical bug on Backward.
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 29 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
16:39
Para revisar, la parte que faltaba de rename inputs/outputs
16:39
https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/581
#581 feat(workflow-manager-core): add support for tasks with renames
Description
Add support for tasks with renames
Motivation and Context
Feature completeness of the MVP
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 29 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
10:20
Para revisar https://github.com/Tvlport-SPC/ts-communication-layer/pull/720
#720 fix(non-air-modify): fix mask min sec docs step test function
Description
Fix mask min sec docs step test function:
• It almost always returned a false positive in the test function.
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
Tvlport-SPC/ts-communication-layer | 30 ene. | Añadido por GitHub
:marca_de_verificación_blanca:
1



Cristian Lorenzo
10:51
Follow-up de la de arriba, subiendo el paquete de graphql que estaba mal: https://github.com/Tvlport-SPC/fes-non-air-modify/pull/121
#121 fix: update dependencies to prevent wrong masking of min sec docs
Description
Update dependencies to prevent wrong masking of min sec docs
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)
• Package dependencies have been updated as much as possible (for minor and patches for sure)
Types of changes
What kind of change does this PR introduce?
• Bug fix
• New feature
Mostrar más
Comments
1
Tvlport-SPC/fes-non-air-modify | 30 ene. | Añadido por GitHub


Cristian Lorenzo
12:08
Tenemos en la colección de Postman como añadir pasajeros con JSON API?


Cristian Lorenzo
12:14
Necesitaré despues de las 16:00 un cable para mirar eso. Si no, pues hago un script y los añado desde SPC, pero prefiero hacerlo directo para evitar indirecciones en el spike.



6 respuestas
Última respuesta hace 9 mesesVer hilo


David Vazquez
12:16
tengo alguna colección de JSON API para añadir pasajeros, luego lo podemos mirar


Cristian Lorenzo
12:18
Genial. A ver si eso funciona, si no kk de la vaca


Adrian Lopez
:prohibido_el_paso: 15:40
https://teamtravelport.slack.com/archives/C05MCC69HK2/p1738247531419879



Ferran Garriga
@canal we are facing some Github service degradation, JFYI
https://www.githubstatus.com/
image.png

image.png
Hilo en accessui-azure-infrastructure | 30 ene. | Ver el mensaje
:+1:
1

15:40
por si veis que github esta que no chuta


Cristian Lorenzo
15:44
Cuando tengáis la oportunidad, miramos lo del JSON API


David Vazquez
15:54
si quieres lo miramos
15:54
a ver si me acuerdo de como funcionaba el postman


Cristian Lorenzo
15:54
Vale, por mi sí


David Vazquez
15:55
llama


Adrian Lopez
:prohibido_el_paso: 09:47
Para revisar: https://github.com/Tvlport-SPC/fes-plugin-settings/pull/84
#84 fix: add schema missing for GET plugin-settings
Description
This PR adds a not contemplated case where a custom MIME GET fetch onto the general settings of a user would not return the default schema.
Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-43165
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/fes-plugin-settings | 31 ene. | Añadido por GitHub


Jose Martin
09:48
tienes una línea comentada


Adrian Lopez
:prohibido_el_paso: 09:49
oh
09:49
vale voy a quitarla jajaja
09:49
ah no
09:49
si es la de la linea 52 es porque es un comentario sobre que tipo utiliza (ya que es javascript y no se pueden usar tipos, es solo como referencia)


Cristian Lorenzo
11:15
Lo de Iberia es urgentillo
11:15
Como no estoy el lunes, mejor que se ponga alguien que no sea yo


David Vazquez
11:16
ok, ya me pongo yo que estoy de shield y dejo de lado el defect que estoy haciendo ahora


Adrian Lopez
:prohibido_el_paso: 09:43
Recordad (por si acaso, que a mi me ha pillao asi de imprevisto) que han hecho los database refresh asi que si os sale que vuestra cuenta esta bloqueada, id a Galileo Desktop y en modo Terminal haced el SON/(usuario de login a host)/UNLOCK, con el password siendo "4TESONLY", no el password que se usa (editado)
:manos_levantadas:
1



Manuel Moraga
12:03
y esta call, farewell pasiona colleagues?


Jose Martin
12:03
para hacer un poco el paripé


Manuel Moraga
12:03
acabo de verlo
12:04
ya lo que faltaba es que encima nos vendan la moto y nos hagan hablar ahí, o sea, has currado bien, no tenemos problemas contigo, te vas a la calle, celebrémoslo
12:04
jajaja


Pedro Morata Vila
12:04
no es paa broma, lo se.
12:05
Cuantos han caido de momento de pasiona?


Manuel Moraga
12:05
nosotros hemos sido 8
12:05
todos los que estamos en remoto
12:05
sobre todo peña en Andalucía y Madrid
12:05
a algunos no los conozco


Pedro Morata Vila
12:05
porque si sois "pocos", entiendeme, quiza hagan sorpreson y salga lista de nominados de pasiona
12:06
esto es todo una fiesta de mal gusto.
12:06
(no solo el farewell, sinó los 3 meses que llevamos)


Joan Perez camprubi
:eat: 12:06
es de todo menos inteligente por su parte


David Vazquez
12:07
bueno hay un chaval de Pasiona de Jaén que todavía sigue no?


Pedro Morata Vila
12:07
no les queda mas remedio, la verdad, ya que en españa y otros paises ormales, hay proteccion al trabajador y hay procesos que hacer.


Joan Perez camprubi
:eat: 12:07
en teoria eran todos los que estan en remoto


David Vazquez
12:07
Javier Moguel creo que se llama


Pedro Morata Vila
12:07
pero podrian simplemente haberlo anunciado todo a la vez


Manuel Moraga
12:07
lo que a mi me han vendido es que nos hemos ido todos los que estamos en remoto, e informalmente, desde travelport, que el motivo era ese, que ser remoto es más caro


Joan Perez camprubi
:eat: 12:08
remoto mas caro?


Manuel Moraga
12:08
pero vaya, que ya da igual


Pedro Morata Vila
12:08
se paga el bit de streaming


Manuel Moraga
12:08
que no importa lo que digan o no
12:08
ya en este punto, me da igual
12:09
paso generalmente de las mierdas corporativas, pero encima una mierda corporativa porque nos echan sin motivo y con mentiras
12:09
es para partirse


David Vazquez
12:09
al final somos solo números para ellos


Manuel Moraga
12:14
no tengo muchas ganas de ir a esa call, no me interesa, para que?


Jose Martin
12:14
ya... yo no iría, la verdad


Manuel Moraga
12:14
ya se acabó, hay que pensar en el próximo reto, seguir creciendo, y reclamar a esta gente lo que es mío, que ya está en proceso
:corazón:
1
:músculo:
1



Joan Perez camprubi
:eat: 12:15
total
12:15
encima tendras que hacer el paripé, de he estado a gusto todo este tiempo noseque
12:15
eso con el equipo que ya lo has hecho que al final es la gente con la que trabajas
12:15
yo tampoco iría ni harto vino
12:15
jaaj


Manuel Moraga
12:15
pero que con sus mentiras, a una persona más debíl la podrían hundir facilmente, vaya han demostrado que son unos cabr... y todo lo que venden es mentira
12:18
el mismo viernes, una jefa de rrhh de pasiona, sube a linkedin un artículo sobre como han ayudado a una trabajadora anónima (jajaja) que se sentía mal en su puesto y no era feliz
12:18
y quedan super guay


Adrian Lopez
:prohibido_el_paso: 12:19
esa trabajadora:
image.png

image.png


3 respuestas
Última respuesta hace 9 mesesVer hilo


Joan Perez camprubi
:eat: 12:19
Pero bueno al final es la condición humana, en todo tipo de relaciones


Manuel Moraga
12:19
luego te ves esta serie, severance, y te dices que no estamos tan lejos


Joan Perez camprubi
:eat: 12:20
Cuando las cosas van bien todo muy bonito, cuando no cada uno barre pa su casa
12:20
Solo hace falta ver, los momentos de crisis y guerras y postguerras, la gente se comería al hijo de su vecino


Pedro Morata Vila
14:40
todo esto sobra 100% que momento mas incomodo


Jose Martin
14:42
total


Joan Perez camprubi
:eat: 14:42
puff


Adrian Lopez
:prohibido_el_paso: 14:43
ha sido bastante incomodo que nadie sabia que decir exactamente


Pedro Morata Vila
14:43
es que......
14:43
creo que NUNCA he visto que se haga una reunion así.
14:44
vale que los compis son remotos, y es imposible hacerlo presencial.... pero madre mia. Manu, la que te has ahorrado


Jose Martin
14:44
al pobre Kofi no le dieron ni tiempo a despedirse
14:44
eso fue heavy también


Joan Perez camprubi
:eat: 14:44
ya ves
14:44
totalmente de acuerdo


Manuel Moraga
14:47
si, me preguntó Laura y le dije que sin ánimo de ofender a los compañeros no me sentía bien, que hay buena gente en travelport pero que esto sobra
14:47
y que esa buena gente, no es la que toma las decisiones


Manuel Moraga
14:47
que no iba a la call

1 respuesta
hace 9 mesesVer hilo


Manuel Moraga
14:48
me comprendió, creo que no he sido el único que no ha ido


Jose Martin
14:48
casi nadie de los 8 ha ido creo


3 respuestas
Última respuesta hace 9 mesesVer hilo


Jose Martin
14:48
tiene todo el sentido


Manuel Moraga
14:49
Antonio ya me había dicho que no iba y con la jugada de pasiona no hay nadie con ánimos


Jose Martin
14:49
pensábamos que iría el Ernest y ni eso


2 respuestas
Última respuesta hace 9 mesesVer hilo


Manuel Moraga
14:55
eso ya habría sido el colmo de falta de respeto
14:55
ya me había dicho Laura que era algo de compañeros, que no vendría esta gente


Jose Martin
15:00
ánimo Manu
:corazón:
1



Israel Naranjo
17:52
mañana vais a la oficina ?


Jose Martin
17:53
si


Joan Perez camprubi
:eat: 17:53
yes


Israel Naranjo
17:53
guay


Adrian Lopez
:prohibido_el_paso: 18:01
yiep


Adrian Lopez
:prohibido_el_paso: 08:04
al final me quedare en casa hoy, que he pasado una noche algo complicada y me noto algo febril


Pedro Morata Vila
09:31
yo me he conseguido despertar hace poco.
Madre mia, vaya noche. ando algo atontado, pero funcional


Jose Martin
15:06
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/574
#574 fix(plugin-traveler-information-gui): change membership number section title
Description
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 | 4 feb. | Añadido por GitHub
15:06
es una chorrada para cambiar un título del R&C
:marca_de_verificación_blanca:
1

15:07
(https://travelport.atlassian.net/browse/EV-43544)
EV-43544 Misleading verbiage for adding membership numbers
Status: To Do
Type: Defect
Assignee: Jose Martin
Priority: 3 - Medium
More actions...
Añadido por Jira Cloud


Pedro Morata Vila
15:21
chicos.
necesitamos mirarnos el servicio de fes-traveler-information cuando haciendo un delete la llamada al cs-traveler-information devuelve algo que no debiera
15:23
@Chema Martin, @Israel, estuvimos metidos en esto, quien podria ponerse conmigo en algun momento para hacer ese fix?
Imagino que haciendo un test especifico y usado el mock que devuelve el servicio deveria ser rápido


Israel Naranjo
15:28
despues de la reunion te parece??


Pedro Morata Vila
15:32
ok!
15:32
gracias!


Adrian Lopez
:prohibido_el_paso: 16:11
en horta mejor salir de ahi que la habitacion se desmutea sola


Jose Martin
16:11
si, es una mierda. Se ve nuestra cámara? no, no?


Adrian Lopez
:prohibido_el_paso: 16:11
sep
16:11
se os ve
16:12
image.png

image.png


David Vazquez
17:24
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/590
#590 fix(plugin-sell): remove comparation price on book and hold success
Description
Related with Jira defect https://travelport.atlassian.net/browse/EV-43502, the second getPrice call from bookAndHoldConfirmationSuccess has been removed.
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 3 feb. | Añadido por GitHub
:marca_de_verificación_blanca:
1



David Vazquez
17:48
mañana va a alguien a la ofi?


Jose Martin
17:48
moi


Adrian Lopez
:prohibido_el_paso: 17:49
yop


Joan Perez camprubi
:eat: 18:15
Nois
18:15
No hemos puesto objetivos esta semana
18:15
Estaría bien mirarlo mañana en la daily


David Vazquez
18:29
los objetivos se ponen el lunes, no el penúltimo día. Aparte yo creo que esta semana pocas cosas se han sacado y completado y por tanto el viernes igual habría que trabajar, pero bueno es mi opinión. (editado)
:+1:
2



David Vazquez
09:55
https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/590
#590 fix(plugin-sell): remove comparation price on book and hold success
Description
Related with Jira defect https://travelport.atlassian.net/browse/EV-43502, the second getPrice call from bookAndHoldConfirmationSuccess has been removed.
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
Comments
1
Tvlport-SPC/pos-web-plugin-sell | 3 feb. | Añadido por GitHub


Jose Martin
17:55
Se puede revisar: https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/278
#278 feat(reservation-details-sync-engine): email and phone conciliation
Conciliation for both email and phone fields.
Description
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 6 feb. | Añadido por GitHub


Adrian Lopez
:prohibido_el_paso: 08:52
Para revisar: https://github.com/Tvlport-SPC/fes-session-manager/pull/92
#92 refactor(fes-session-manager): move ACCESSGROUP_OVERRIDE from spc-infra-configmaps to fes-session-manager
Description
This PR moves the key ACCESSGROUP_OVERRIDE from spc-infra-configmaps to fes-session-manager; allowing us to do proper maintenance of configmaps that are only used in a specific service.
The current task is also linked to this other PR: Tvlport-SPC/spc-infra-configmaps#107
Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43315
• 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/fes-session-manager | 5 feb. | Añadido por GitHub


Jose Martin
10:34
He arreglado el pequeño bug mientras estábamos en la daily, ya se puede mirar de nuevo: https://github.com/Tvlport-SPC/pos-web-plugin-reservation-details/pull/278
#278 feat(reservation-details-sync-engine): email and phone conciliation
Conciliation for both email and phone fields.
Description
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
Comments
1
Tvlport-SPC/pos-web-plugin-reservation-details | 6 feb. | Añadido por GitHub
:marca_de_verificación_blanca:
1



David Vazquez
15:15
PR para revisar https://github.com/Tvlport-SPC/pos-web-ticket/pull/413
#413 fix(issue-ticket): manage irrecoverable errors for issue ticket requests
Description
Manage irrecoverable errors for issue ticket flow displaying a new component IssueTicketErrorIrrecoverableScreen when the end-transaction fails and the retrievePnr gets empty.
image
Is this change related to a Jira Task?
• Yes
• Story Link: https://travelport.atlassian.net/browse/EV-43356
• 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
Tvlport-SPC/pos-web-ticket | 7 feb. | Añadido por GitHub
https://github.com/Tvlport-SPC/pos-web-ticket/pull/413

:marca_de_verificación_blanca:
1



David Vazquez
09:27
podéis echarle un vistazo a la PR? merci


Israel Naranjo
10:31
cuando os arrejunteis para el postman añadirme a la llamada please


Joan Perez camprubi
:eat: 14:26
Quien va mañana a la ofi?


Adrian Lopez
:prohibido_el_paso: 14:26
yo voy


David Vazquez
14:37
yo no, esta semana estoy en Valencia


Jose Martin
14:37
yo iré también


Joan Perez camprubi
:eat: 14:38
ok
14:38
guay si somos mas de 2 vengo también


Pedro Morata Vila
16:07
yo no estoy seguro


Adrian Lopez
:prohibido_el_paso: 17:06
Para revisar Parte 1: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/578
Lo de la India; respecto al requirement desde el inicio y el extensionId desapareciendo
:marca_de_verificación_blanca:
1



Adrian Lopez
:prohibido_el_paso: 17:26
Para revisar Parte 2: https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/603
Lo de la India también, pero es el añadir el Arrival
:marca_de_verificación_blanca:
1



Jose Martin
17:28
segundas partes nunca fueron buenas...
17:28
menos el padrino II


Adrian Lopez
:prohibido_el_paso: 17:28
y menos Shrek 2


Jose Martin
17:30
no se hablaba que lo del pcc de la india no era necesario?
17:30
al final sí?


Adrian Lopez
:prohibido_el_paso: 17:30
sep, al final si
17:31
el requirement acaba siendo (origen india || final india) && pcc indio


Jose Martin
17:31
que toca pelotas llegan a ser, ay mamita
17:31
ok


Adrian Lopez
:prohibido_el_paso: 17:31
y lo de la parte 1 es porque el react hook forms borra todo parametro que no este vinculado a un input
17:32
y he aplicado lo que se ha hecho en el SARA y en una parte de hoteles
17:32
poner un input hidden vinculado a ello
:+1:
1



Jose Martin
17:33
esto va al payload del submit?


Adrian Lopez
:prohibido_el_paso: 17:33
sep


Jose Martin
17:33
lo del extensionId me refiero
17:33
ok


Israel Naranjo
17:41
vaya guarradas


Adrian Lopez
:prohibido_el_paso: 17:41
lo recomiendan desde el github de react-hook-forms mismo asi que
17:42
mejor esto a lo que tenia pensado que era re-asignar el id en el onSubmit (editado)


Israel Naranjo
17:43
es una ñapa teneer que andar con inventos
17:43
react kk


David Vazquez
17:56
para revisar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/602
#602 refactor(plugin-sell): add extensions to persisted travelers
Description
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-plugin-sell | 10 feb. | Añadido por GitHub
17:56
1ª parte tambien


David Vazquez
10:08
otra mas para revisar https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/580
#580 feat(plugin-traveler-information-gui): change conditional for readonly traveler name fields
Description
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 | 11 feb. | Añadido por GitHub
10:08
podéis echarle un vistazo a las dos PR's? Merci


Israel Naranjo
11:16
nos arrejuntamos para lo del postman??? @aquí


Joan Perez camprubi
:eat: 11:17
image.png

image.png
11:17
:alegría: Isra guarrete


Jose Martin
11:17
jajaja


Israel Naranjo
11:18
tengo falta de cariño :beso_de_corazón:


Joan Perez camprubi
:eat: 11:18
para cuando el anillo?
11:18
rompecorazones


Israel Naranjo
11:18
:muriéndose_de_risa:


Jose Martin
11:19
estamos David y yo, si quiere entra Isra


Israel Naranjo
11:21
si porf
11:22
pasa el link please


Jose Martin
11:24
vale, pero con uno de los dos que estemos ya estaría


Israel Naranjo
11:24
:sudor:


Jose Martin
11:25
llamo


Pedro Morata Vila
11:25
me uno


Israel Naranjo
11:26
:muriéndose_de_risa: perdon
11:28
me podeis meter de nuevo en la llamada que la lie...


Israel Naranjo
11:37
par revisar -> https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/577
:marca_de_verificación_blanca:
1



Joan Perez camprubi
:eat: 12:50
Avisad y vamos al freshkito
12:50
:stitch_hungry:


Adrian Lopez
:prohibido_el_paso: 15:18
Para super fast as heck revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/582 (editado)
#582 style(plugin-traveler-details): changed india gst text from Agency to…
… Company
Description
This PR changes texts from Agency to Company, all according recent reunions on updating the UX.
Motivation and Context
JIRA: https://travelport.atlassian.net/browse/EV-43687
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 | 11 feb. | Añadido por GitHub
:marca_de_verificación_blanca:
2

15:18
Solo tiene texto (editado)


Adrian Lopez
:prohibido_el_paso: 16:22
Para revisar: https://github.com/Tvlport-SPC/pos-web-traveler-details/pull/583/files
Reimplementa otra vez la funcionalidad antigua del requirement de la India (editado)


Jose Martin
16:24
y eso?


Adrian Lopez
:prohibido_el_paso: 16:26
era un cambio a ultima hora que no me di cuenta que habia que implementar
16:26
:this-is-fine-fire:


David Vazquez
16:34
osea que no es siempre required no? solo cuando se rellena un campo como antes


Adrian Lopez
:prohibido_el_paso: 16:34
sasto


Joan Perez camprubi
:eat: 17:39
Pa que lo revisen https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/606


David Vazquez
17:40
me resulta un poco confuso el nombre de las FF
17:41
parecen lo mismo, que ambas muestren el modal


Joan Perez camprubi
:eat: 17:41
ah ya
17:41
en la definicion del ticket dice que para el link se use la antigua que dice Modal
17:41
y para el modal pues he creado esa nueva que dice modal tambien


David Vazquez
17:41
ufff


Joan Perez camprubi
:eat: 17:41
que propones?
17:42
el lio es usar la antigua para lo del link
17:42
pero bueno
17:42
si asi lo ha pedido nuria que es la que controla de las ff
17:42
pues ya esta bien


David Vazquez
17:42
a alguna le quitaría la palabra modal si lo que se quiere es no mostrar el modal en una y en otra si (editado)


Joan Perez camprubi
:eat: 17:42
al final son los managers que activan y desactiban eso
17:42
mientras ellos se aclaren
17:42
yo habría hecho lo mismo
17:42
una ponerle LINK
17:42
y otra MODAL
17:43
lógico no?


David Vazquez
17:43
sí el algo así


Joan Perez camprubi
:eat: 17:43
pero bueno eso ha pedido Nuria


David Vazquez
17:43
entonces la que pone WITH_MODAL no muestra el modal no? (editado)
17:43
y la otra que pone MODALS es la que lo muestra, verdad?


Joan Perez camprubi
:eat: 17:43
eso es


David Vazquez
17:43
:alegría:


Joan Perez camprubi
:eat: 17:43
que hi farem


David Vazquez
17:44
se lo has comentado lo de que es confuso y te ha dicho que lo dejes así igualmente?
17:44
si es así pues palante


Joan Perez camprubi
:eat: 17:45
no
17:45
pero lo pone en el ticket
17:45
usar la que hay existente para lo del LINK
17:45
y tiene la palabra MODALS
17:45
así que ya es sabedora de eso
17:45
de que una FF que dice MODALS servira para LINK


David Vazquez
17:46
a ver se lo voy a comentar
17:46
puede ser que no haya caido en eso


Joan Perez camprubi
:eat: 17:47
:awesome:
17:48
pero aver una cosa
17:48
esto ya estaba definido asi
17:48
y si ahora se quiere cambiar hay que ir a varios repos a cambiar el naming de la FF
17:48
de verdad lo ves necesario?
17:48
si esto ya era así
17:50
y hay muchas FF que no son descriptivas, no se para que complicar una cosa cuando ya se ha definido
17:50
si es solo una clave, que una vez implementada la ponen en true y nadie mas se acuerda de eso
17:50
que si fuera una nueva lo entiendo, pero es una que ya existe y que esta en true por defecto


David Vazquez
17:53
hombre pero las FF tienen que ser un poco descriptivas con lo que van a hacer, sino luego es un puto lio saber que hacen cuando ya no te acuerdas de la implementación (editado)


Joan Perez camprubi
:eat: 17:53
por eso
17:53
que no lo era ya
17:54
la cuestion es la nueva FF
17:54
esa no te parece descriptiva?
17:54
nadie ha dicho que haya que modificar la antigua
17:54
que me quieres meter mas curro o que cabron?
17:54
si quieres cambio la nueva
17:54
a eso si le veo sentido


David Vazquez
17:54
la que existía era descriptiva porque hacía eso, mostrar el modal, pero si ahora esa no se quiere que muestre el modal no debería incluir la palabra digo yo


Joan Perez camprubi
:eat: 17:54
bueno no
17:55
porque hacia eso y mas cosas


David Vazquez
17:55
ya he hablado con Nuria


Joan Perez camprubi
:eat: 17:55
pero si lo pone ella especificamente en el ticket
17:55
dejar la antigua para el link y crear una nueva para el modal


David Vazquez
17:55
dice que no hay problema con cambiar la que ya existía, que sí que es confuso
17:55
supongo que no era consciente de ello


Joan Perez camprubi
:eat: 17:56
eso es
17:56
que no hay problema
17:56
pero tampoco que sea necesario


David Vazquez
17:56
lo que ponen en la tarea no siempre va a misa Joan


Joan Perez camprubi
:eat: 17:56
xD
17:56
ou mama


David Vazquez
17:59
no te lo digo porque tenga ganas de fastidiar, es que estoy seguro que cualquier otro dirá lo mismo. Estoy seguro que si se lo pasas a Arya te dirán exactamente lo mismo, que es confuso y les chirria el nombre, independientemente de lo que ponga en la tarea que hayan creado para ello.


Joan Perez camprubi
:eat: 18:01
bueno
18:01
no te preocupes


David Vazquez
18:01
ya lo he hablado con Nuria y está de acuerdo, ya está no hay que darle más vueltas


Joan Perez camprubi
:eat: 18:01
...


David Vazquez
09:18
Buenas, habéis visto el comentario de Kike de que revisemos las PR's de dependabot en nuestros repositorios?


Adrian Lopez
:prohibido_el_paso: 09:19
sep, ha puesto dos ejemplos pero supongo que habran que replicarlos en los otros repos tambien no?


David Vazquez
09:19
sí, habrá que echar un vistazo a todo lo que sea nuestro


Israel Naranjo
09:21
lo mirare (editado)


Joan Perez camprubi
:eat: 09:29
Para revisar y aprobar https://github.com/Tvlport-SPC/pos-web-plugin-sell/pull/606








David Vazquez
09:42
imagino que el split de la FF vendrá en otra PR no? porque esa solo cambia el nombre de la existente


Joan Perez camprubi
:eat: 12:13
A revisar:
https://github.com/Tvlport-SPC/pos-web-plugin-air-confirmation/pull/184
https://github.com/Tvlport-SPC/pos-web-plugin-air-price/pull/289


Jose Martin
15:42
Joder, lo de aceptar las PR's casi sin leerlas nos lo tenemos que hacer mirar eh?
15:43
O seré yo que leo muy lento, pero casi siempre me queda más de la mitad por leer y ya está aceptada por el equipo y mergeada :cara_con_ceja_levantada:


Joan Perez camprubi
:eat: 15:43
jaja
15:43
me paso igual


Israel Naranjo
15:43
si lo pillas mas tarde que otro q lo esta revisando
15:43
es normal


Jose Martin
15:44
que va! si acababan de pasar la PR


Joan Perez camprubi
:eat: 15:44
toda la razón
15:44
los ninja son rapidos


Israel Naranjo
15:44
a mi me han pasado por particular el link a 3.15


Joan Perez camprubi
:eat: 15:44
amigo
15:44
a veces hay cosas que no las vemos


Israel Naranjo
15:45
preguntar antes de hablar hay q hacerselo mirar


Joan Perez camprubi
:eat: 15:45
jajaja


Israel Naranjo
15:45
luego vienen las discursiones


Jose Martin
15:45
hombre, no podía saber eso


Israel Naranjo
15:46
se pregunta


Jose Martin
15:46
lo digo de buen rollo igualmente


Israel Naranjo
15:46
oki error miio
15:46
:sonrisa_con_sudor:


Jose Martin
15:46
yo creo que tenemos que hacer un team building para relajar tensiones
15:46
en un par de semanas nos vamos a dar de hostias los unos a los otros jajaja


Israel Naranjo
15:47
par nada q soy el mas bajito


Jose Martin
15:47
que es normal, la situación es una mierda
Cargando historial...
Últimos mensajes


Enviar un mensaje a rumiko_developers