gisce / fiber-diagram

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

Afegir nom als cables de fibra #20

Closed mameijeiras closed 6 months ago

mameijeiras commented 1 year ago

Resum dels canvis necessaris a nivell de cable de FO:

mguellsegarra commented 1 year ago

Com ho veieu?

@ecarreras @mameijeiras

Screenshot 2023-06-13 at 12 17 54

ecarreras commented 1 year ago

Em fa por, que moltes vegades no hi cabrà la descripció

mameijeiras commented 1 year ago

Em fa por, que moltes vegades no hi cabrà la descripció

I amb l'id del cable + un tooltip a sobre?

mguellsegarra commented 1 year ago

Com vosaltres cregueu, penso que la descripció sencera potser hi ha cops que no hi capiga, però la id del cable + tooltip potser és bona

mameijeiras commented 6 months ago

@ecarreras Faig un 🆙 a aquesta issue, sembla que ja tenim clar quin serà el disseny final. Avui hem tingut reunió i hem dedicit que a nivell de cable només s'ha de mostrar la posició (camp position que encara hem d'afegir) i que la resta d'informació es pot treure (amb els colors és suficient).

La posició servirà per ordenar els cables dins de cada CTO (de la mateixa manera que tenim previst que funcioni amb els Patch Panels) i contindrà un tooltip amb el nom del cable (camp name).

Afegeixo la informació al missatge principal amb un resum dels canvis, qualsevol cosa aquí estem! @mguellsegarra

mguellsegarra commented 6 months ago

@mameijeiras ho he estat mirant... et sembla si de moment, fins que no despleguis el tema d'afegir el camp position, faig que si els tubs i fibres no tenen el paràmetre, faci fallback a la id, com es mostrava fins ara?

mameijeiras commented 6 months ago

@mguellsegarra el camp position com a molt tard el tindrem divendres, t'aviso quan ho despleguem a telecos

mguellsegarra commented 6 months ago

@mameijeiras va bé però que fins que no despleguis el tema d'afegir el camp position, faig que si els tubs i fibres no tenen el paràmetre, faci fallback a la id, com es mostrava fins ara?

L'altre tema que em pregunto és els "wires", que fins ara no hem mostrat mai cap referència a ells,... seguim així ?

mameijeiras commented 6 months ago

@mguellsegarra no cal que mostri res perquè és informació que per a l'usuari no li aporta res.

Precisament el canvi fa referència als "wires" o cables, el camp position és a nivell de wire :smile:

A nivell de tubs i fibres no s'ha de fer res, l'ordre és sempre tal i com arriba.

mguellsegarra commented 6 months ago

@mameijeiras

@mguellsegarra no cal que mostri res perquè és informació que per a l'usuari no li aporta res.

Però comentaves això:

Avui hem tingut reunió i hem dedicit que a nivell de cable només s'ha de mostrar la posició (camp position que encara hem d'afegir)

Mostrem el position amb tooltip amb el nom a nivell de wires o no mostrem res?

mameijeiras commented 6 months ago

@mguellsegarra sí això sí, el que no s'han de mostrar són els ids de tubs i fibres :rocket:

mameijeiras commented 6 months ago

@mguellsegarra com que ens estem fent un "lio" he actualitzat la issue, espero que hagi quedat més clar, si no fem trucada en un moment

mguellsegarra commented 6 months ago

@mameijeiras

tooltips

mameijeiras commented 6 months ago

Mola @mguellsegarra! :clap:

mguellsegarra commented 6 months ago

@mameijeiras Vigent a 3.0.0-alpha.35, ja em dieu :)

mameijeiras commented 6 months ago

@mameijeiras Vigent a 3.0.0-alpha.35, ja em dieu :)

Provat i bé, però entenc que amb l'id/position és suficient, ara mateix surt "Cable 1": image

mguellsegarra commented 6 months ago

Això és perquè no té position el wire i per això fa fallback, en aquest cas al name, que li devies haver posat "Cable 1", "Cable 2".

Quan tinguis el tema d'afegir el position es veurà tal com t'ensenyava al gif d'abans

mameijeiras commented 6 months ago

Entesos @mguellsegarra :raised_hands:

mguellsegarra commented 6 months ago

@mguellsegarra el camp position com a molt tard el tindrem divendres, t'aviso quan ho despleguem a telecos

@mameijeiras es pot provar ja el tema del position amb els canvis desplegats?

mameijeiras commented 6 months ago

@mguellsegarra aquesta tarda m'hi poso, t'aviso quan estigui! :rocket:

mameijeiras commented 6 months ago

@mguellsegarra ja està disponible el camp position a l'entorn de Telecos :rocket: Però sembla que no acaba d'anar bé: image

mguellsegarra commented 6 months ago

Pel que veig el camp de posició arriba a false dins del JSON que treu l'ERP, és normal?

El codi està preparat perquè el camp position sigui numèric.

mameijeiras commented 6 months ago

@mguellsegarra és un tema de Python, arriba booleà false. Prefereixes que no arribi res? Quin format et va millor?

mguellsegarra commented 6 months ago

@mguellsegarra és un tema de Python, arriba booleà false. Prefereixes que no arribi res? Quin format et va millor?

Res, era saber-ho. He modificat el component perquè fagi un sanitize a les dades d'entrada i ignori els camps position que no son numèrics.

Està disponible a v3.0.0-alpha.42 ;)

mameijeiras commented 6 months ago

Genial! :star_struck:

mguellsegarra commented 6 months ago

Passat a main, disponible a la Release - v2.2.9