Small mismatch in user info from /proposal vs /person
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Área
- api, backend-api-design
Línea de trabajo
Comienza rastreando los puntos de entrada /v1/proposal/{id} y /v1/person/username/{username} que generan las respuestas mostradas. Compara sus campos de usuario y determina la nomenclatura y disponibilidad previstas por el proyecto para first_name, last_name, orcid e institution. Se considera completado cuando las respuestas de API relacionadas exponen de forma coherente los campos acordados y hay pruebas que cubren los cambios.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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
- Lenguaje dominante
- Python
- Estrellas
- 0
- Forks
- 13
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Sin guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de NSLS2/nsls2api
-
Raise BNLPeople exceptions for callers to process or propagatePosiblemente ocupada @nmaytan la tomó hace 17 días. Abierto
-
Review outstanding TODOsAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 30/100
-
Dificultad 3/5 1-2 días Aptitud para principiantes 72/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
Todos los issues de NSLS2/nsls2api
Issues similares
-
bug needs-triage priority:medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
yeongseon/azure-functions-logging-python#583 ·
Los mantenedores suelen responder en 1 día
-
documentation needs-triage priority:medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
yeongseon/azure-functions-validation-python#546 ·
Los mantenedores suelen responder en 1 día
-
PHP workflow streaming requests buffer the full SSE responsePosiblemente ocupada @Winoren la tomó hoy. Abiertoreview: medium
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
langgenius/dify#44270 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
P4: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
jeffknupp/association#523 ·
-
bug help wanted
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
DarkStarSystems/pcons#229 ·
Los mantenedores suelen responder en 1 día