Contract errors and unclarity about Pokemon form names
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 30/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Área
- api, documentation
Línea de trabajo
Comienza comparando el contrato de PokemonDetail para held_items y los campos nullable requeridos con las respuestas de Delphox y otros Pokémon afectados. Después, inspecciona el comportamiento de búsqueda de nombres para aegislash, basculegion y basculin; se considera terminado cuando el contrato coincide con las respuestas reales y los nombres de las formas excepcionales están documentados.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
During testing of your OpenAI contract solution, I have come across some errors.
In the contract, it is stated this:
(Within PokemonDetail in contract)
held_items:
type: object <---- held_items must be an object.
required:
- item
- version_details
However, for example Delphox (https://pokeapi.co/api/v2/pokemon/delphox/), held_items is an empty array, which fails in parsing because it is not an object. In some cases, an attribute is required by contract but is null. These two cases produces these two errors:
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Not a JSON Object: []
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Not a JSON Object: null
A large number of the total pokemon in your API have one or both of these problems. (About ~100 from my incomplete testing) All my testing was done purely with methods generated from the contract with OpenAPI Generator, and when Pokemon were queried by name - not ID. I have not tested with ID query.
Also, I found out through my testing that there are a few pokemon who cannot be queried with "normal" name means. I saw in earlier issues that you are aware, but it would be helpful to have this information accessible in the PokeAPI with the actual list of Pokemon who don't fit the normal bill and need form-appendixes to their name.
For example:
- aegislash
- basculegion
- basculin
- Lenguaje dominante
- Python
- Estrellas
- 5.4k
- Forks
- 1.2k
- Merge medio
- 2 d 12 h
- PR fusionados (30 d)
- 18
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 PokeAPI/pokeapi
-
Blank past ability entries Abierto
Dificultad 2/5 1-2 días Aptitud para principiantes 76/100
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
openapi integers range. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 55/100
-
UV hardlink files Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
-
Ball legality Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
Todos los issues de PokeAPI/pokeapi
Issues similares
-
agent-ready documentation needs-triage
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
-
documentation
Dificultad 1/5 Menos de una hora Aptitud para principiantes 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
-
Add https://search.jeremyh.xyz/ Abiertoinstance instance add
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
searxng/searx-instances#939 · 1 comentario ·
-
area-deployment area-integrations triage:bot-seen
Dificultad 2/5 Medio día Aptitud para principiantes 86/100