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

Supporto alle carriere multiple #13

Open
intersimone999 opened this issue Nov 20, 2015 · 9 comments
Open

Supporto alle carriere multiple #13

intersimone999 opened this issue Nov 20, 2015 · 9 comments
Assignees

Comments

@intersimone999
Copy link
Contributor

Si potrebbe aggiungere il supporto alle carriere multiple.

In particolare, secondo me bisognerebbe accettare un parametro opzionale nelle richieste con l'identificativo della carriera: se non è specificato, si intende la carriera più recente (per la retro-compatibilità).

@intersimone999
Copy link
Contributor Author

Servirebbe anche un servizio "getCareers", che invia la lista di carriere dell'utente.

@mattmezza
Copy link
Contributor

si ma come si impostano le diverse carriere su esse3? Esse3 come le gestisce? Non credo siano dei semplici parametri in url... qualcuno conosce qualcuno che può verificare?

@intersimone999
Copy link
Contributor Author

Io ora ho due carriere. Da una prima analisi sembra che basti un id nell'url, anche perché non ci sono cookie che registrano la carriera. Le informazioni sono registrate in sessione penso.

@mattmezza
Copy link
Contributor

E' facile estrarre gli id delle carriere? Hai provato a confermare la tua teoria tramite semplici chiamate http tipo con postman?

@intersimone999
Copy link
Contributor Author

Gli ID sono semplicemente negli URL della pagina di selezione delle carriere, estrarli è banale. Noi comunque anche adesso facciamo una chiamata del genere per chi ha carriere multiple, solo che prendiamo sempre la prima carriera nella tabella. Basta aggiungere il parametro opzionale alle richieste, usarlo per le chiamate di selezione della carriera e implementare il servizio per ricevere la lista di carriere disponibili. Posso occuparmi io di questo 👍

@mattmezza
Copy link
Contributor

Ottimo! Per qualsiasi cosa fai un fischio

@intersimone999
Copy link
Contributor Author

Fatto!

@intersimone999
Copy link
Contributor Author

Il login non funziona di nuovo con le mie credenziali, quindi credo che ci sia di nuovo il problema con le carriere multiple. Appena riesco verifico.

@mattmezza
Copy link
Contributor

Porco il ca**o. API di merda.

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

No branches or pull requests

2 participants