Skip to content

Nuova colonna Merce nell'elenco delle Attivita' - #1927

Open
Andrea-Bruno wants to merge 1 commit into
devcode-it:masterfrom
Andrea-Bruno:fix/colonna-merce-1825
Open

Andrea-Bruno wants to merge 1 commit into
devcode-it:masterfrom
Andrea-Bruno:fix/colonna-merce-1825

Conversation

@Andrea-Bruno

Copy link
Copy Markdown

Ciao e grazie per il progetto.

Ho letto la issue #1825 e ho preparato una correzione.

La richiesta era di avere, nell'elenco delle Attivita', una colonna che mostri la merce collegata a ogni attivita', cosi' da vedere subito quali prodotti entrano in quell'intervento senza doverlo aprire.

La correzione aggiunge questa nuova colonna. Per ogni riga dell'elenco viene mostrato l'elenco dei prodotti dell'attivita', ognuno con la sua quantita' e la sua descrizione, uno sotto l'altro. La colonna e' anche ricercabile, quindi si puo' filtrare l'elenco cercando un prodotto.

Il modo in cui l'ho ottenuta e' una query che raccoglie i prodotti di ciascuna attivita' al momento della lettura dell'elenco, senza appesantire la query principale e senza toccare le altre colonne. Ho scritto la colonna come sottovista collegata alla singola attivita', seguendo lo stesso schema gia' usato per altre colonne dello stesso elenco.

Essendo una colonna nuova, l'ho inserita nella migrazione della versione 2.12.0, che e' quella in preparazione, con il titolo sia in italiano sia in inglese.

Registers a new Merce view for the Interventi module that lists, for each activity, the goods linked to it with quantity and description. Added as a correlated subquery so the main list query is untouched, and shipped in the 2.12.0 migration with Italian and English titles.
@Andrea-Bruno

Copy link
Copy Markdown
Author

Hello. The Codacy check on this pull request reports one new issue of at least minor severity, but the detail is not visible to us from outside Codacy, and the check output is empty. The change is a single SQL migration that adds a new searchable column to the activities list, following the same zz_views pattern already used by the other columns in that same list. Could a maintainer let us know which line Codacy is flagging and what it wants changed, so we can fix it? We are glad to adjust it. Thank you.

This branch has not been deployed

No deployments
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.

1 participant