Skip to content

Snake/camel case conversion in label generation#241

Merged
mnamici merged 5 commits intoobdasystems:developfrom
MariaRosariaFraraccio:snake-camel-case-conversion
Dec 27, 2022
Merged

Snake/camel case conversion in label generation#241
mnamici merged 5 commits intoobdasystems:developfrom
MariaRosariaFraraccio:snake-camel-case-conversion

Conversation

@MariaRosariaFraraccio
Copy link
Contributor

No description provided.

@mnamici
Copy link
Collaborator

mnamici commented Dec 20, 2022

Le modifiche che abbiamo elencato durante l'ultimo meeting sono complete qui?
Se sì vorrei fare un refactor della componente grafica questa settimana e poi fare il merge.

@MariaRosariaFraraccio
Copy link
Contributor Author

Sì, c'è tutto

@mnamici
Copy link
Collaborator

mnamici commented Dec 20, 2022

Ok grazie, allora procedo con il refactor dell'interfaccia e aggiungo qui.

@mnamici mnamici force-pushed the snake-camel-case-conversion branch from 6a08845 to 90c6edf Compare December 22, 2022 21:44
@mnamici
Copy link
Collaborator

mnamici commented Dec 22, 2022

Fatto il rebase che prima era basato sulla vecchia versione della PR per il label wrap automatico.

@mnamici
Copy link
Collaborator

mnamici commented Dec 23, 2022

Ok iniziando a fare la review mi sono accorto che ci sono numerose cose che non vanno bene nell'interfaccia dell'ontology manager, e di conseguenza impatta sulle modifiche qui.

Ad esempio ho notato che le preferenze sulla label automatica dalla IRI sono impostate con un comando, il che significa che di fatto viene vista come una modifica del progetto di cui puoi fare undo!

Da un occhiata al seguente comportamento:
bug

Se non bastasse che mi fa fare undo i setter si sono pure inventati di invertire i valori delle altre impostazioni che non ho toccato.

Questo ci dice che dobbiamo tenere un'attimo da conto la logica di conversione camel/snake case ma prima di applicarla dobbiamo sistemare l'interfaccia. Faccio una PR separata, poi una volta sistemato il manager applichiamo lì la logica di trasformazione snake e camel case.

@mnamici mnamici force-pushed the snake-camel-case-conversion branch from 90c6edf to d17cf7f Compare December 27, 2022 10:38
@mnamici mnamici merged commit c18019b into obdasystems:develop Dec 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Automatizzare trasformazioni tra snake/camel case e spaziatura normale nelle label

2 participants