Compilazione Checklist e Pull Request

PieroDal commented 4 weeks ago

Salve, dove trovo l'informazione per riempire la colonna "Timestamp" della Checklist ? Per caricare il risultato del test ho letto che bisogna fare una pull request ma anche leggendo la documentazione non riesco a capire come fare, prostete aiutarmi con esempi semplici ?

vigliottim commented 4 weeks ago

Buonasera, può riempire la colonna "Timestamp" della checklist inserendo il valore contenuto in eventDate e associato alla chiamata al servizio di validazione per lo specifico test. Può recuperarlo chiamando, ad esempio, il servizio status:

Servizio di Recupero Stato Transazione per TraceId: https://github.com/ministero-salute/it-fse-support/tree/main/doc/integrazione-gateway#11-servizio-di-recupero-stato-transazione-per-traceid
Servizio di Recupero Stato Transazione per WorkflowInstanceId: https://github.com/ministero-salute/it-fse-support/tree/main/doc/integrazione-gateway#10-servizio-di-recupero-stato-transazione-per-workflowinstanceid

Di seguito le alleghiamo dei link con indicazioni relative alla creazione della pull-request:

Documentazione pull-request: https://github.com/ministero-salute/it-fse-accreditamento#pull-request
Pull-request: https://github.com/ministero-salute/it-fse-support/issues/695#issuecomment-2022834466
Esecuzione fork e commit: https://github.com/ministero-salute/it-fse-support/issues/853#issuecomment-2124107085

Rimaniamo a disposizione per ulteriori chiarimenti.

vigliottim commented 3 weeks ago

Buonasera, non avendo ricevuto ulteriori riscontri, procediamo alla chiusura della presente issue. Potrà riaprirla nel caso in cui fosse necessario ulteriore supporto. Grazie.

PieroDal commented 3 weeks ago

Buona sera, grazie per la risposta una domanda riguardo la checklist nel file word ci sono 6 casi test OK (da 0 a 5) ma nel file excel (versione 8.2.1) parte da 1 a 5, lo 0 quindi non và compilato, giusto ? Nella sezione degli esempi xml https://github.com/ministero-salute/it-fse-accreditamento/tree/main/Test%20Case/Validazione/Documenti%20XML%20Casi%20OK/5%20-%20Casi%20OK%20Referto%20di%20Medicina%20di%20Laboratorio il contenuto degli XML non corrisponde a quanto scritto nel file doc. Ad esempio, per il caso 1 nel file word si parla di esame microbiologico delle vie aeree ma nell'esempio xml c'è ALBUMINA_URINE. Come mi devo comportare ?

vigliottim commented 3 weeks ago

Rispondiamo in linea alle sue domande:

riguardo la checklist nel file excel ci sono 6 casi test OK (da 0 a 5) ma nel file excel parte da 1 a 5, lo 0 quindi non và compilato, giusto ?

Il caso di test 0 non è descritto poiché prevede esclusivamente gli elementi obbligatori secondo HL7 Italia. E' necessario compilarlo poiché è sempre applicabile.

Nella sezione degli esempi xml https://github.com/ministero-salute/it-fse-accreditamento/tree/main/Test%20Case/Validazione/Documenti%20XML%20Casi%20OK/5%20-%20Casi%20OK%20Referto%20di%20Medicina%20di%20Laboratorio il contenuto degli XML non corrisponde a quanto scritto nel file doc. Ad esempio, per il caso 1 nel file word si parla di esame microbiologico delle vie aeree ma nell'esempio xml c'è ALBUMINA_URINE. Come mi devo comportare ?

I file di esempio sono stati inseriti a scopo dimostrativo. Dovrà seguire le specifiche contenute nei file riportati nella cartella TestCase/Validazione/LAB per eseguire correttamente i casi di test.

Rimaniamo a disposizione per chiarimenti.

PieroDal commented 2 weeks ago

Buona sera, dopo aver ottenuto la validazione dei cinque casi OK (Referto di Laboratorio), ho cominciato a vedere i casi KO ma ho dei dubbi.

Nel file Word (casi di test LAB.docx) dice di fare un test non compilando il confidentialityCode e così ho fatto ma la validazione và comunque a buon fine ottenendo il TraceID, il workflowInstanceId ecc. Vi allego il pdf con all'interno il CDA ed il log Cda e Log Mancata compilazione confidentialityCode.zip

Seguendo, invece, il file excel accreditamento-checklist_V8.2.1.xlsx, il test successivo che riguarda la sezione LAB è ID 28 (token jwt mancante di campi obbligatori). Devo seguire solo quanto scritto nella sezione Summary, cioè i test con ID 28,36,44,52,53,54,55,56,57,58,59,60,61,62 ?

vigliottim commented 1 week ago

Buongiorno, il caso di test 6 (KO) prevede di non inserire l'informazione sul livello di riservatezza del campo confidentialityCode:

Nel CDA2 da lei utilizzato da lei, pertanto, non valorizzando solo l'attributo code, come indicato di seguito:

As is: <confidentialityCode code="" codeSystem="" codeSystemName="HL7 Confidentiality" displayName="Normal" />

To be: <confidentialityCode code="" codeSystem="2.16.840.1.113883.5.25" codeSystemName="HL7 Confidentiality" displayName="Normal" />

otterrà un errore sintattico come da caso di test:

Per l'esecuzione del caso di test con ID = 28 può seguire le indicazioni riportate in "DESCRIZIONE CASO DI TEST":

Rimaniamo a disposizione per ulteriori chiarimenti.

PieroDal commented 1 week ago

buongiorno, quel <confidentialityCode code="" codeSystem="" codeSystemName="HL7 Confidentiality" displayName="Normal" /> con anche il code system vuoto, è stato il mio secondo tentativo. ho ottenuto la validazione anche come proposto da voi <confidentialityCode code="" codeSystem="2.16.840.1.113883.5.25" codeSystemName="HL7 Confidentiality" displayName="Normal" /> ed ho anche provato ad eliminare tutto il "rigo" confidentialityCode ma il sistema continua a validare: {"traceID":"b1dc75495859b1ed","spanID":"b1dc75495859b1ed","workflowInstanceId":"2.16.840.1.113883.2.9.2.120.4.4.46a41df0ab0514f11c0811056832c3225e06c8e11824f27c7e5517ca5cfc57fe.cc6331d9f4^^^^urn:ihe:iti:xdw:2013:workflowInstanceId"}

PieroDal commented 1 week ago

un'altra cosa. Devo seguire i casi scritti nel file Word (casi di test LAB.docx) o quelli del file excel (accreditamento-checklist_V8.2.1.xlsx) ?? Chiedo poichè i casi di test sono differenti e perchè, per esempio, l'errore di sintassi relativo confidentialityCode non c'è nel file excel.

vigliottim commented 1 week ago

buongiorno, quel <confidentialityCode code="" codeSystem="" codeSystemName="HL7 Confidentiality" displayName="Normal" /> con anche il code system vuoto, è stato il mio secondo tentativo. ho ottenuto la validazione anche come proposto da voi <confidentialityCode code="" codeSystem="2.16.840.1.113883.5.25" codeSystemName="HL7 Confidentiality" displayName="Normal" /> ed ho anche provato ad eliminare tutto il "rigo" confidentialityCode ma il sistema continua a validare: {"traceID":"b1dc75495859b1ed","spanID":"b1dc75495859b1ed","workflowInstanceId":"2.16.840.1.113883.2.9.2.120.4.4.46a41df0ab0514f11c0811056832c3225e06c8e11824f27c7e5517ca5cfc57fe.cc6331d9f4^^^^urn:ihe:iti:xdw:2013:workflowInstanceId"}

Le chiediamo di effettuare una nuova prova di validazione inserendo l'elemento confidentialityCode come indicato: <confidentialityCode code="" codeSystem="2.16.840.1.113883.5.25" codeSystemName="HL7 Confidentiality" displayName="Normal" /> e inviarci la response completa ottenuta. Grazie.

vigliottim commented 1 week ago

un'altra cosa. Devo seguire i casi scritti nel file Word (casi di test LAB.docx) o quelli del file excel (accreditamento-checklist_V8.2.1.xlsx) ?? Chiedo poichè i casi di test sono differenti e perchè, per esempio, l'errore di sintassi relativo confidentialityCode non c'è nel file excel.

Il file word Casi-di-test-LAB.docx contiene la descrizione dei casi di test OK e KO presenti nella checklist. Può seguire le specifiche indicate sui relativi file excel, CDA2_RefertoMedicinadiLaboratorio_OK.xlsx e CDA2_RefertoMedicinadiLaboratorio_KO.xlsx, per costruire il CDA2 necessario per svolgere correttamente il caso di test in esame. Per tutti gli altri casi di test che si riferiscono, ad esempio, a timeout o ai JWT, può far riferimento alla colonna "DESCRIZIONE CASO DI TEST" della checklist.

PieroDal commented 1 week ago

niente da fare, ecco qui: {"traceID":"aee49b5d99c38ed9","spanID":"aee49b5d99c38ed9","workflowInstanceId":"2.16.840.1.113883.2.9.2.120.4.4.46a41df0ab0514f11c0811056832c3225e06c8e11824f27c7e5517ca5cfc57fe.2a9435177c^^^^urn:ihe:iti:xdw:2013:workflowInstanceId"} Validate.log RefertoDEC_CDA.pdf

LucaRogledi commented 1 week ago

Buongiorno, dalle verifiche effettuate sulla chiamata con traceId:

"traceID":"aee49b5d99c38ed9"

Risulta che il cda utilizzato contiene:

<confidentialityCode code="N" codeSystem="2.16.840.1.113883.5.25" codeSystemName="HL7 Confidentiality" displayName="Normal"/>

La invitiamo a verificare il pdf utilizzato per la chiamata, effettuando una nuova prova seguendo le correzioni indicate precedentemente. Dalle nostre verifiche tramite la chiamata di validazione utilizzando il pdf da lei riportato RefertoDEC_CDA.pdf si ottiene "Errore di sintassi." Grazie

PieroDal commented 1 week ago

Grazie per la segnalazione che ha fatto venir fuori un errore!

Buongiorno, dalle verifiche effettuate sulla chiamata con traceId:

"traceID":"aee49b5d99c38ed9"

Risulta che il cda utilizzato contiene:

<confidentialityCode code="N" codeSystem="2.16.840.1.113883.5.25" codeSystemName="HL7 Confidentiality" displayName="Normal"/>

La invitiamo a verificare il pdf utilizzato per la chiamata, effettuando una nuova prova seguendo le correzioni indicate precedentemente. Dalle nostre verifiche tramite la chiamata di validazione utilizzando il pdf da lei riportato RefertoDEC_CDA.pdf si ottiene "Errore di sintassi." Grazie

Grazie per la segnalazione. In effetti c'era un errore sul file che veniva realmente inviato.

LucaRogledi commented 1 week ago

Buonasera, ringraziamo per il feedback, e procediamo con la chiusura della presente issue. Si può procedere con la riapertura nel caso fosse necessario ulteriore supporto. Grazie

PieroDal commented 1 week ago

Un'altra domanda. Per i casi KO che vengono prevenuti dall'applicativo, tipo mancanza di parte dell'anagrafica, sesso, ecc, posso impostare la NON applicabilità ?

LucaRogledi commented 1 week ago

Buonasera, confermiamo che i casi di test che non possono essere svolti devono essere indicati come non applicabili. Ricordiamo, inoltre, che qualora l'applicabilità sia NO, è necessario compilare anche la colonna Razionale di applicabilità, al fine di motivare la non applicabilità del caso di test

PieroDal commented 1 week ago

Di seguito le alleghiamo dei link con indicazioni relative alla creazione della pull-request:

* _Documentazione pull-request:_ https://github.com/ministero-salute/it-fse-accreditamento#pull-request

* _Pull-request:_ [Prima pull request #695 (comment)](https://github.com/ministero-salute/it-fse-support/issues/695#issuecomment-2022834466)

* _Esecuzione fork e commit:_ [Problemi per creazione repository per accreditamento #853 (comment)](https://github.com/ministero-salute/it-fse-support/issues/853#issuecomment-2124107085)

Rimaniamo a disposizione per ulteriori chiarimenti.

Buonasera, sto cercando di seguire la procedura di caricamento risultati ma non riesco a creare il repository.

Se clicco su https://github.com/ministero-salute/it-fse-accreditamento appare la seguente videata

immagine

e forse a seguito di altri tentativi sbagliati, se clicco su Create New Fork (come da documentazione) appare la seguente videata

immagine

come devo proseguire ?

LucaRogledi commented 6 days ago

Buongiorno, L'errore da lei riscontrato potrebbe essere dovuto dal fatto che il repository è gia stato generato da una fork precedente. Pertanto suggeriamo di:

Aprire il repository forkato https://github.com/PieroDal/it-fse-accreditamento se conferma essere il repository da lei forkato precedentemente.
Partendo dal repository indicato (se le modifiche erano gia state applicate a questo repository, assicurarsi di avere un backup offline per evitare di perderle) effettuare una Sync come indicato nel seguente commento https://github.com/ministero-salute/it-fse-support/issues/794#issuecomment-2095963176

verificare che le modifiche desiderate siano presenti o applicarle nuovamente tramite commit.
cliccare su contribute - open pull request

assicurarsi che base sia ministero-salute/it-fse-accreditamento e head repository il repository forkato contenente le nuove modifiche.

aggiungere la descrizione e cliccare su Create Pull Request.

Grazie

PieroDal commented 6 days ago

grazie

Cliccando sul link [https://github.com/PieroDal/it-fse-accreditamento] appaiono files che non mi riguardano (anche perchè non sono riuscito a caricare ancora nulla)

immagine

C'è un modo per partire da zero ? Oppure modificare/eliminare la fork/repository ?

LucaRogledi commented 6 days ago

Salve, è corretto che siano presenti i file da lei riportati nello screenshot, in quanto, a questi andranno ad aggiungersi i dati che desidera caricare. Può procedere caricando i risultati da lei ottenuti all'interno della cartella GATEWAY seguendo quanto indicato in documentazione - Procedura di caricamento dei risultati per quanto riguarda la struttura corretta.
Grazie

PieroDal commented 6 days ago

Ma in questa fork, creata per errore, non c'è il percorso corretto GATEWAY/A1#111#DECSISTEMISRLX1/DEC_Sistemi_Srl/LABNET/1.4 ( la barra al contrario và messa davanti al punto, giusto ?? ). Non posso/riesco ad eliminarla ma posso fare L'Upload dei file, ho provato con uno a caso (Status_CT1.log) e me l'ha consentito ma non riesco/posso creare le cartelle (p.e. files) Mi consente anche di creare la Pull Request ma non ho proseguito per evitare di fare altri "casini". Che faccio ?

LucaRogledi commented 5 days ago

Buongiorno, Riportiamo di seguito una segnalazionein cui sono riportati gli step da seguire per la creazione del struttura necessaria per l'accreditmaneto e il caricamento dei file: #853 .

Lasciamo inoltre gli step da seguire in caso volesse eliminare il repository forkato per errore:

Aprire il repository forkato che desidera eliminare
Cliccare impostazioni

Nella schermata settings - general aperta scorrere fino alla fine e cliccare delete this repository

Grazie

PieroDal commented 5 days ago

Grazie, repository eliminato e ricreato correttamente (ho scorperto che bisogna dare i nomi delle cartelle uno alla volta, p.e. gateway\ e solo dopo proseguire un passo alla volta, io invece prima facevo il copia ed incolla dell'intero percorso). Ho caricato anche i file al posto giusto ma mi sono accorto che uno di questi, giusto il data.json, aveva un errore all'interno. Come faccio a cancellare il file e rifare l'upload di quello corretto ?

LucaRogledi commented 5 days ago

Buongiorno, Per eliminare il file desiderato può utilizzare la sezione indicata per esempio:

Effettuare successivamente il commit

Ottenendo il seguente risultato

Riportiamo anche la documentazione github in caso possa essere d'aiuto https://docs.github.com/en/repositories/working-with-files/managing-files/deleting-files-in-a-repository. Mentre per l'upload del file corretto può trovare l'opzione nella sezione di add file:

Grazie

PieroDal commented 5 days ago

Grazie, probabilmente sbaglio nel modo in cui arrivo ai file poichè da qui: immagine la possibilità di Eliminare non c'è. Qual'è il percorso che devo seguire ?

vigliottim commented 5 days ago

Buon pomeriggio, le chiediamo di controllare se esiste una scrollbar nel menu a tendina e l'opzione "Delete file" è presente in basso, come mostra la seguente figura:

PieroDal commented 5 days ago

sottilissima ma c'era 🤦🏻‍♂️ Grazie ho fatto!

Ho creato anche la PR https://github.com/ministero-salute/it-fse-accreditamento/pull/1705

La richiesta è corretta ?

vigliottim commented 5 days ago

La informiamo che la pull request è stata correttamente creata. La invitiamo a effettuare le seguenti correzioni:

eliminare il file "vuoto.pdf"
rinominare il file json da "Data.Json" a "data.json" Inoltre, la esortiamo a controllare che la tripletta indicata nel json e nella checklist deve essere sempre uguale a quella indicata nel percorso in cui inserisce i file. Riceverà gli esiti delle elaborazioni via e-mail entro qualche giorno lavorativo.

PieroDal commented 5 days ago

Grazie Vuoto.pdf eliminato come anche il file json con le maiuscole. Riguardo alla tripletta, la differenza sta in DEC Sistemi Srl e DEC_SistemiSrl della cartella ? (ho messo "" al posto dello spazio perchè l'ho letto nella documentazione)

Devo modificare la descrizione della PR, il json ed il file excel con i trattini bassi ? Nella richiesta di accreditamento come "subject_application_vendor" è stato definito "DEC Sistemi Srl" (con gli spazi)

vigliottim commented 2 days ago

Buongiorno, l'underscore va inserito solo nel nome della directory di sottomissione. Rimaniamo a disposizione per ulteriori chiarimenti.

PieroDal commented 2 days ago

Buongiorno e grazie, ho modificato la descrizione della Pull Request eliminando gli underscore.

LucaRogledi commented 2 days ago

Buongiorno, Le confermiamo che la pull request è stata presa in carico. Chiudiamo pertanto il presente ticket, dal momento che eventuali aggiornamenti relativi alla richiesta di accreditamento le saranno notificati via email entro qualche giorno lavorativo. Grazie.

PieroDal commented 2 days ago

Buonasera, ho aggiornato il report-checklist, devo effettuare una nuova Pull Request ?

LucaRogledi commented 1 day ago

Buongiorno, Essendo presente una PR aperta da parte sua, non ancora, elaborata è possibile applicare modifiche su quella ( https://github.com/ministero-salute/it-fse-accreditamento/pull/1705 ) come dalle nostre verifiche risulta lei abbia già fatto. Restiamo a disposizione per ulteriore supporto. Grazie

PieroDal commented 1 day ago

Grazie a voi per il prezioso supporto

LucaRogledi commented 1 day ago

Buongiorno, ringraziamo per il feedback e procediamo con la chiusura della presente issue. Grazie

PieroDal commented 12 hours ago

Buongiorno, scrivo solo per avvisarvi (non so se sia necessario) che ho aggiornato il report-checklist

vigliottim commented 12 hours ago

Buongiorno, scrivo solo per avvisarvi (non so se sia necessario) che ho aggiornato il report-checklist

Grazie per la segnalazione. La sua PR è stata correttamente creata. Riceverà l'esito delle elaborazioni via e-mail nel giro di qualche giorno lavorativo.

ministero-salute / it-fse-support

Compilazione Checklist e Pull Request #1187