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

Add ContactPerson tag in metadata (required by Avviso nr. 29) #114

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lrealdi
Copy link

@lrealdi lrealdi commented Oct 14, 2020

No description provided.

@peppelinux
Copy link
Member

Ciao, come mai questo contributo non viene unito?

@simevo
Copy link
Collaborator

simevo commented Feb 12, 2021

Ciao scusate, innanzitutto grazie per i contributi !

Teniamo d'occhio questo repo ma possiamo lavorarci solo nei ritagli di tempo ... per accettare una PR vorremmo dedicarci il tempo giusto quindi non sempre si riesce a incastrare.

Prometto un ritorno a breve !

@khekkosan
Copy link

Ciao ragazzi! vi ringrazio intanto per il lavoro che state facendo..volevo chiedervi se avreste modo di accettare questo contributo..nel frattempo noi stiamo usando il repo dei ragazzi di OpenContent per andare avanti con l'autoassessment..

..penso però che sarebbe importante usare il vendor ufficiale! (senza nulla togliere ai ragazzi di OpenContent che ringrazio)

'sp_contact_ipa_code' => 'test',
'sp_contact_fiscal_code' => 'XXXXXX00X00X000X',
'sp_contact_email' => '[email protected]',
'sp_contact_phone' => '12345678910',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Da correggere il possibile warning per le validazioni con italia/spid-sp-test

The //ContactPerson[@contactType='other']/TelephoneNumber element MUST start with "+39"

Copy link
Collaborator

@lucacracco lucacracco left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Al netto della segnalazione (warning) questa PR la porterei avanti perchè necessaria per la validazione e il funzionamento.

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.

5 participants