Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[BUG] Parameters should be case insensitive #1179

Open
pedwards95 opened this issue Dec 25, 2024 · 2 comments
Open

[BUG] Parameters should be case insensitive #1179

pedwards95 opened this issue Dec 25, 2024 · 2 comments

Comments

@pedwards95
Copy link

pedwards95 commented Dec 25, 2024 •
edited
Loading

Steps to Reproduce:

  1. Submit a request with capitals. eg. https://pokeapi.co/api/v2/pokemon/Iron-Crown
  2. Receive no reply (404)

Requests with capitals should still work. Very easy fix just to lowercase parameters.
@Naramsim
Copy link
Member

Hi! Could you try and implement the fix?

@pedwards95
Copy link
Author

pedwards95 commented Dec 26, 2024 •
edited
Loading

Yeah I can put together a PR when I get back from vacation.

Edit: This will be delayed if it comes from me.
Edit2: Update: After reviewing data during integration, and in relation to numerous asinine choice like the use of hectograms, and the refusal to fix it as seen in this ticket #1182
I will not be fixing this or any of the other issues encountered.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Naramsim @pedwards95