Contract errors and unclarity about Pokemon form names
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 30/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Lĩnh vực
- api, documentation
Hướng nghiên cứu
Bắt đầu bằng cách so sánh contract của PokemonDetail đối với held_items và các trường nullable bắt buộc với các response của Delphox và những Pokémon bị ảnh hưởng khác. Sau đó kiểm tra hành vi tra cứu tên đối với aegislash, basculegion và basculin; được coi là hoàn thành khi contract khớp với các response thực tế và tên của các form ngoại lệ đã được ghi lại.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
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
- Ngôn ngữ chính
- Python
- Star
- 5.4k
- Fork
- 1.2k
- Merge trung bình
- 2 ngày 12 giờ
- Pull request đã merge (30 ngày)
- 18
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của PokeAPI/pokeapi
-
Blank past ability entries Đang mở
Độ khó 2/5 1-2 ngày Mức phù hợp với người mới 76/100
-
bug
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
-
openapi integers range. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 55/100
-
UV hardlink files Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
-
Ball legality Đang mở
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
Tất cả issue của PokeAPI/pokeapi
Issue tương tự
-
agent-ready documentation needs-triage
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 88/100
-
documentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 91/100
-
workflow-status page template still says reusable workflows are "triggered only by workflow_call:" Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 92/100
-
Add https://search.jeremyh.xyz/ Đang mởinstance instance add
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
searxng/searx-instances#939 · 1 bình luận ·
-
area-deployment area-integrations triage:bot-seen
Độ khó 2/5 Nửa ngày Mức phù hợp với người mới 86/100