gisce / fiber-diagram

Component to show and interact with fiber connections
0 stars 0 forks source link

Desapareixen les connexions quan es connecten fibres alineades #19

Closed mameijeiras closed 9 months ago

mameijeiras commented 1 year ago

El problema és que a part que desapareixen les línies, no es poden desconnectar les fibres i s'ha de refrescar la pàgina. Si es desa, apareix igual. Adjunto captures de com ho he reproduït a l'entorn de Telecos Test (primera captura) i en local (segona captura).

El comportament és lleugerament diferent en local, ja que en local quan connectes dues fibres es crea la bola de la connexió més gran, no sé si es tracta d'un cas previst però entenc que s'haurien de mostrar sempre les connexions per tal de poder connectar i desconnectar les fibres @ecarreras

Enregistrament de pantalla des de 12-6-23 12:48:02.webm

Enregistrament de pantalla des de 12-6-23 12:48:24.webm

mguellsegarra commented 1 year ago

He estat revisant el component un cop migrat a vite, i em funciona bé a l'entorn

https://beta.webclient.telecos.gisce.dev/ @mameijeiras pots fer-hi una ullada per si ho pots reproduïr?

El https://webclient.telecos.gisce.dev/ queda pendent de fer el merge per a que utilitzi el nou component.

mameijeiras commented 1 year ago

He estat revisant el component un cop migrat a vite, i em funciona bé a l'entorn

https://beta.webclient.telecos.gisce.dev/ @mameijeiras pots fer-hi una ullada per si ho pots reproduïr?

El https://webclient.telecos.gisce.dev/ queda pendent de fer el merge per a que utilitzi el nou component.

@mguellsegarra ho he pogut reproduir a l'entorn beta:

Enregistrament de pantalla des de 13-6-23 12:58:23.webm

mameijeiras commented 1 year ago

@mguellsegarra amb el primer CTO es pot veure fàcilment agafant els dos primers ports de dalt

ecarreras commented 1 year ago

El problema és quan es pensa que vols fer un "tub a tub" perquè totes les fibres d'un costat les fusiones amb totes les de l'altre costat, però el tub de l'altre costat té més fibres.

image

mguellsegarra commented 1 year ago

Crec que té a veure amb la build que es fa al webclient, ja que amb npm start no passa... ho investigo

mguellsegarra commented 1 year ago

Per fi he pogut reproduïr el que comentaveu, per si us passa mai, tenia extensions de bloqueig d'anuncis i trackers que feien que no pogués clicar depenent de en quina fibra tocava... Investigo ara sí què fa malament amb el tema d'ajuntar els tubs.

mguellsegarra commented 1 year ago

Llest, ja he vist què era.

He pujat un fix a develop, amb el que des de l'entorn de beta haurieu de poder provar-ho com ja funciona correctament:

@ecarreras @mameijeiras

Screenshot 2023-06-15 at 15 17 53

Screenshot 2023-06-15 at 15 18 52

ecarreras commented 1 year ago

Testejat a develop i funciona bé! 🚀

mameijeiras commented 1 year ago

Moltes gràcies, @mguellsegarra!

image

mguellsegarra commented 1 year ago

@ecarreras @mameijeiras Us sembla si ho passo a main?

ecarreras commented 1 year ago

Si si, i tant

mguellsegarra commented 1 year ago

Ara estava fent proves, i crec que no es poden colapsar mai els tubs, vaig a revisar-ho abans de passar res a main.

mguellsegarra commented 1 year ago

He pujat una nova versió a l'entorn de develop, ara ja es col·lapsen els tubs quan vagin totes les fibres a un altre tub i coincideixin en número.

@ecarreras @mameijeiras Si voleu fer una prova ràpida per assegurar i ho passem a main

mameijeiras commented 1 year ago

@mguellsegarra ho he provat al CTO B i no funciona del tot bé, sembla que no es poden desconnectar els cables:

Enregistrament de pantalla des de 16-6-23 11:03:25.webm

ecarreras commented 1 year ago

Jo crec que ja va guai! 🚀 image

mguellsegarra commented 1 year ago

Passat a main ! 💪

mameijeiras commented 1 year ago

Passat a main ! muscle

Genial @mguellsegarra! Quan puguis revisa el que passa al CTO B, no es pot desfer una connexió tub a tub, suposo que deu estar relacionat.

mguellsegarra commented 1 year ago

Quan puguis revisa el que passa al CTO B, no es pot desfer una connexió tub a tub, suposo que deu estar relacionat.

Ara ho reviso, crec que té a veure amb la detecció de canvis

mguellsegarra commented 1 year ago

@mameijeiras

Ja he trobat què era. He pujat un fix a develop (entorns beta), per a que hi feu una ullada i un cop ho vegeu bé si voleu podem passar-ho a main.

Ja em dieu, merci! :)

cc: @ecarreras

mameijeiras commented 1 year ago

@mameijeiras

Ja he trobat què era. He pujat un fix a develop (entorns beta), per a que hi feu una ullada i un cop ho vegeu bé si voleu podem passar-ho a main.

Ja em dieu, merci! :)

cc: @ecarreras

@mguellsegarra Segueixo sense poder desfer la connexió al CTO B 😢

mguellsegarra commented 1 year ago

@mguellsegarra Segueixo sense poder desfer la connexió al CTO B 😢

Que estrany, ho acabo de provar i em funciona, @mameijeiras

ctob

mameijeiras commented 1 year ago

@mguellsegarra Ara m'ha funcionat també!

mguellsegarra commented 1 year ago

@ecarreras @mameijeiras Us sembla si ho passo a main?

ecarreras commented 1 year ago

Per mi perfecte 👍🏻

mameijeiras commented 9 months ago

@mguellsegarra @ecarreras aquesta issue ja es pot tancar, oi?

mguellsegarra commented 9 months ago

Sí, feel free!