ministero-salute / it-fse-support

https://ministero-salute.github.io/it-fse-support/
34 stars 20 forks source link

Pull request #486

Closed DATASOFTNA closed 1 year ago

DATASOFTNA commented 1 year ago

Buongiorno, chiedo scusa se vi disturbo per una questione banale, ma non riesco a creare una pull request dopo avere aggiornato il file report-checklist.xlsx. Ci deve essere qualche passaggio che mi sfugge, perche' vedo le due richieste fatte in una lista, ma non credo siano stati effettuate. Potreste darmi una mano ? Grazie.

DATASOFTNA commented 1 year ago

Buongiorno, chiedo scusa se vi disturbo per una questione banale, ma non riesco a creare una pull request dopo avere aggiornato il file report-checklist.xlsx. Ci deve essere qualche passaggio che mi sfugge, perche' vedo le due richieste fatte in una lista, ma non credo siano stati effettuate. Potreste darmi una mano ? Grazie.

Ho fatto qualche altro passaggio un po' a casaccio :) , ma sembra che la pull request sia stata effettuata, nel caso puo' chiudere la issue. Grazie e scusi per il disturbo.

p.s. Mi e' giunta una mail credo di conferma della richiesta di pull, ma il file mi sembra quello di prima. Devo aspettare o c'e' qualcosa che (di nuovo) ho sbagliato ? Grazie.

IStacchiotti commented 1 year ago

Buongiorno @DATASOFTNA, Confermiamo che la pull è stata correttamente aperta su GitHub (https://github.com/ministero-salute/it-fse-accreditamento/pull/856). Vi segnaliamo però che il file "report-checklist.xlsx" non è stato inserito nel branch dedicato alla vostra azienda/applicativo (GATEWAY/A1#111DATASOFTNA/DATASOFT). Potreste verificare?

In caso di difficoltà, vi consigliamo di chiudere questa pr e ripetere la procedura dall'inizio seguendo le indicazioni contenute a questo link su Slack. Rimaniamo a disposizione per eventuali chiarimenti. Grazie

DATASOFTNA commented 1 year ago

Purtroppo slack mi dice che il mio browser non e' adatto inoltre ho di nuovo editato il file ed avrei necessita' di creare una nuova pull request per l'aggiornamento 5 del file. E' da tempo che non utilizzo github e non mi ricordo piu' come fare delle operazioni. In maniera molto poco sofisticata, potrebbe indicarmi come fare ad emettere la nuova pull request partento da questa pagina ? https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0 Purtroppo non trovo al momento un browser che vada a genio al canale slack ed ho anche perso le credenziali per l'ingresso.

DATASOFTNA commented 1 year ago

Vi segnaliamo però che il file "report-checklist.xlsx" non è stato inserito nel branch dedicato alla vostra azienda/applicativo (GATEWAY/A1#111DATASOFTNA/DATASOFT). Potreste verificare?

Purtroppo non saprei neppure come verificare. Io sono convinto che la mia cartella dedicata sia in https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0 , ma potrei sbagliarmi. Pero' mi sembra che il file che e' li' dentro sia quello che mi aspetto.

DATASOFTNA commented 1 year ago

Sono riuscito a trovare browser e credenziali e sono entrato nel canale slack. Forse sono io che opero in maniera errata, ma mi e' sembrato un marasma peggio di questo, non saprei neanche come sottoporre la mia richiesta.

IStacchiotti commented 1 year ago

Adesso la sua Pull Request si presenta come riportato nello screen allegato:

image

Le confermiamo che risulta aggiornato il file che è stato inserito correttamente nel vostro branch, tuttavia dovreste provvedere all'eliminazione, tramite nuova commit, del file esterno evidenziato in rosso nell'immagine, per consentirci di procedere con il merge della pr e le successive analisi. Grazie mille

DATASOFTNA commented 1 year ago

Per cortesia, potrebbe dirmi qual e' la cartella sulla quale devo operare ? Io ne trovo due, una e' https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0 L'altra e' https://github.com/DATASOFTNA/it-fse-accreditamento Su quale devo caricare i file ed eseguire la richiesta di pull ? Per quanto riguarda la videata precedente, non so se ho fatto bene, ma ho "chiuso" la pull request. Intanto ho aggiornato il file report, dato che FSE mi ha mandato una mail nella quale chiede degli aggiornamenti. Chiedo scusa del disturbo che vi sto arrecando, ma ho completamente dimenticato il funzionamento di questo sito.

IStacchiotti commented 1 year ago

Buongiorno @DATASOFTNA, Le due cartelle in questione fanno parte entrambe della sua copia del main del ministero del ministero della salute. In questa copia, dovreste aggiornare sempre la cartella di vostra competenza, che identifica l'applicativo per il quale intende chiedere l'accreditamento (quindi in questo caso, se gli aggiornamenti che intende riportare si riferiscono a LABGUI 3.0, dovrebbe caricare i file in https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0). Tramite l'apertura di una pull request si fa richiesta di riportare tali aggiornamenti sul main del ministero della salute; questo aggiornamento è finalizzato tramite il merge della pull request, previsto in una fase intermedia del processo di accreditamento. La invitiamo a non chiudere mai in autonomia la pull request, a meno che non ci sia un errore o intenda risottometterla (le pull request in stato "Closed" vengono annullate e non considerate ai fini dell'accreditamento), ma piuttosto a lasciarla aperta: la pull request verrà mergiata a completamento di un set di verifiche preliminari.

Le confermiamo infine che la #861 è stata aperta correttamente, le chiediamo solo la cortesia di eseguire una commit correttiva per rimuovere il file esterno, segnalato nel messaggio precedente, che risulta ancora presente. Di seguito riportiamo uno screen per indicarle il file al quale facciamo riferimento rispetto alla vostra copia del main.

Speriamo di aver chiarito i suoi dubbi, rimaniamo a disposizione in caso sia necessari chiarimenti ulteriori.

image
DATASOFTNA commented 1 year ago

Grazie, adesso la situazione mi e' un po' piu' chiara. Il file indicato l'ho cancellato, spero di avere risolto la questione e attendo il responso dell'analisi da parte di FSE.

IStacchiotti commented 1 year ago

Di seguito uno screen della situazione sulla pr #861 in seguito alla sua ultima commit:

image

Sembrerebbe che abbiate eliminato il file contenuto nella cartella dedicata all'applicativo, anzichè quello contenuto esternamente. Potreste verificare e riprovare eventualmente?

La vostra pr dovrebbe presentarsi circa così: image

DATASOFTNA commented 1 year ago

Ho cancellato quello contenuto in

https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0

Se ho sbagliato, come posso porre rimedio ?

IStacchiotti commented 1 year ago

Può eseguire una commit correttiva sulla stessa pr per:

  1. inserire nuovamente il file checklist nella cartella "https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0";
  2. eliminare il file checklist caricato al livello della cartella GATEWAY
DATASOFTNA commented 1 year ago

Ho ricaricato il file in https://github.com/DATASOFTNA/it-fse-accreditamento/tree/main/GATEWAY/A1%23111DATASOFTNA/DATASOFT/LABGUI/3.0

ma in https://github.com/DATASOFTNA/it-fse-accreditamento per quanto ci provi non riesco ad eliminarlo.Ho il dubbio che debba clickare sul pulsante verde commit changes dopo la cancellazione, ma ho il timore di rovinare qualcosa.

IStacchiotti commented 1 year ago

Ok, le confermo che ora vediamo correttamente il file di cui al punto 1. Per quanto riguarda il punto 2., riprovi pure facendo click sul pulsante verde a valle della cancellazione: una volta effettuate le modifiche deve essere eseguito il commit anche in caso di rimozione dei file.

DATASOFTNA commented 1 year ago

Ok, fatto. Grazie mille.

IStacchiotti commented 1 year ago

Ottimo, le confermiamo che la modifica è stata svolta correttamente. Riceverà aggiornamenti in merito all'esito delle verifiche. Procediamo alla chiusura del presente. Grazie