Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Snake/camel case conversion in label generation #241

Merged

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
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 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