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

Il controllo se già associato alla sede è limitato ai soli volontari #481

Merged
merged 1 commit into from Jan 14, 2017
Merged

Il controllo se già associato alla sede è limitato ai soli volontari #481

merged 1 commit into from Jan 14, 2017

Conversation

yakky
Copy link
Member

@yakky yakky commented Jan 14, 2017

Fix #217
Fix #139

Qui è necessario decidere se filtrare per esclusione (cioè escludendo solo le estensioni) o per inclusione (quindi solo per i volontari)

@yakky
Copy link
Member Author

yakky commented Jan 14, 2017

@CroceRossaItaliana/sviluppo è deployata su luke nello stato attuale

@luca-dex
Copy link
Member

Che politica hai adottato?

Le regole da seguire sono quelle che ho indicato su https://github.com/CroceRossaItaliana/jorvik/issues/217#issuecomment-272620864

@yakky
Copy link
Member Author

yakky commented Jan 14, 2017

@luca-dex esattamente quelle: se la persona risulta volontaria nella sede verso cui chiede il trasferiment oquesto viene bloccato nel solito modo. Per le altre condizioni invece viene permesso come hai descritto. Gli automatismi di chiusura delle appartenenze vecchie in caso di approvazione e tutto il resto della logica rimane invariato. In questo caso era solo un controllo supplementare nel form di richiesta

@luca-dex
Copy link
Member

Ok, verifico quando risorge luke che la cosa funzioni correttamente

@luca-dex
Copy link
Member

Io sono volontario in un Comitato e ho un'estensione in un altro. Se provo a cliccare sulla voce "trasferimenti" del menù succede questo:

attributeerror at utente trasferimento

@yakky
Copy link
Member Author

yakky commented Jan 14, 2017

@luca-dex ci dovremmo essere su luke

@luca-dex
Copy link
Member

Mi permette di richiedere il trasferimento su di un comitato su ciu ho estensione in maniera corretta. Merge anche di questa.

@luca-dex luca-dex merged commit b76bacc into CroceRossaItaliana:master Jan 14, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants