"is_default" is not unique per species
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Tranquilo
- Stack tecnológico
- python
- Área
- api, backend-api-design
Línea de trabajo
Comienza con la documentación de PokemonForm y los datos de pokemon.csv mencionados en la issue, y luego sigue cómo la API expone is_default y form_order. Revisa los comentarios existentes antes de establecer la regla de reemplazo. Se considera terminado cuando el comportamiento de la forma primaria está acordado, implementado y documentado, sin dejar ambigua la semántica del campo actual.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
The PokemonForm documentation describes is_default as
True for exactly one form used as the default for each Pokémon.
Of the 249 Pokémon with more than one form in pokemon.csv, 222 of them have more than one form with is_default set to True.
While the easiest fix would just be to purge these secondary is_default forms, I think that field should just be deprecated and replaced with a different method of determining the primary form. I have to imagine this field is not widely used by API consumers, as it would not work for its documented purpose.
My preferred design would be to simply have the form of the species with the first form ID be the primary form. It looks like there isn't strictly a form_id field, but it roughly corresponds to the form_order field. So my suggestion would be to just define that the form of the Pokémon with the smallest value in form_order is considered the default form.
- 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