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

Aggiunta ruolo automatica #58

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Zhigalin
Copy link
Contributor

@Zhigalin Zhigalin commented Oct 9, 2023

La modifica proposta aggiunge un opzione al plugin che permette di settare un ruolo aggiuntivo all'utente che si logga con SPID.
Utile ad esempio per fare una sezione chiusa sul sito dedicata solo ai utenti SPID.

@milesimarco
Copy link
Member

Ciao @Zhigalin e grazie per la PR. Posso chiederti quale sia il caso d'uso per la modifica effettuata e eventuali vantaggi rispetto ad applicare la logica tramite hook quando l'utente viene registrato con Spid?

@Zhigalin
Copy link
Contributor Author

Ciao @milesimarco,
la modifica e' stata sviluppata per un ordine professionale che ci ha chiesto di creare una sezione sul loro sito con dei moduli da compilare e dei documenti da consultare riservata alle persone che accedono con lo SPID.

Un altro caso d'uso potrebbe essere quello di fornire dei privilegi aggiuntivi su un sito sito ad account di cui si e' sicuri che appartengono a una persona vera.

Il vantaggio principale quello di non dover scrivere del codice sul sito del cliente,
che personalmente trovo un pregio importante.
Ulteriore vantaggio e' poter disabilitare la funzionalita o cambiare il ruolo che viene assegnato lato frontend.

@Zhigalin
Copy link
Contributor Author

Sto tra l'altro sviluppando un altra modifica che permetterebbe di abilitare la registrazione su un sito con il primo accesso via SPID come da issue #26 aggiungendolo come una funzionalita che si puo abilitare e configurare nelle impostazioni del plugin, dimmi poi se va bene che faccia la PR anche su quello

@Zhigalin
Copy link
Contributor Author

Zhigalin commented Dec 20, 2023

@milesimarco ciao, faccio solo un piccolo reminder su questa PR

@milesimarco
Copy link
Member

Ciao @Zhigalin
sto facendo alcuni test integrando #58 e #26 abilitando la registrazione e settando il ruolo (mantenendo il naming del tuo campo add_role).

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.

2 participants