Small mismatch in user info from /proposal vs /person
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Ambito
- api, backend-api-design
Direzione di ricerca
Inizia tracciando gli entry point /v1/proposal/{id} e /v1/person/username/{username} che producono le risposte mostrate. Confronta i relativi campi utente e determina la denominazione e la disponibilità previste dal progetto per first_name, last_name, orcid e institution. Il lavoro è completato quando le risposte API correlate espongono in modo coerente i campi concordati e i test coprono le modifiche.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
The "users" key in the /v1/proposal's response is not the same as the response in /person
For example: https://api.nsls2.bnl.gov/v1/proposal/320326
The users section in the url above has my user info:
{
"first_name": "Venkateswaran",
"last_name": "Shekar",
"email": "[email protected]",
"bnl_id": "27196",
"username": "vshekar1",
"is_pi": false,
"orcid": "0000-0001-5625-2475"
}
While the response in: https://api.nsls2.bnl.gov/v1/person/username/vshekar1
{
"firstname": "Venkateswaran",
"lastname": "Shekar",
"email": "[email protected]",
"username": "vshekar1",
"bnl_id": "27196",
"bnl_employee": true,
"institution": "Brookhaven National Laboratory",
"orcid": null,
"globus_username": null,
"pass_unique_id": null,
"account_locked": null,
"cyber_agreement_signed": "2022-08-01T00:00:00",
"facility_code": null,
"facility_name": null,
"citizenship": null
}
Would be good to standardize first_name + last_name key names in both responses. Additionally the orcid is not available in /v1/person. Finally if user "institution" is available in the proposal response, it would be really useful for the lambda project.
Happy to make a PR so address them
- Lingua principale
- Python
- Stelle
- 0
- Fork
- 13
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di NSLS2/nsls2api
-
Raise BNLPeople exceptions for callers to process or propagateForse già presa @nmaytan l’ha presa 9 giorni fa. Aperta
-
Review outstanding TODOsAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
Tutte le issue di NSLS2/nsls2api
Issue simili
-
dependencies feature github_actions good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
wemake-services/wemake-django-template#3149 ·
I maintainer di solito rispondono entro 1 giorno
-
[request] vsg/1.1.16Apertaupstream update
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
conan-io/conan-center-index#31142 ·
I maintainer di solito rispondono entro 1 giorno
-
area:core bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
request-theme
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 70/100
LizardByte/ThemerrDB#8877 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
area/install-update comp/gateway P0 sweeper:risk-compatibility type/bug
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 72/100
NousResearch/hermes-agent#135997 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno