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.

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

update 'get_item_plaintext' URL to the latest version #94

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ruoyu-lu
Copy link

The get_item_plaintext function previously utilized an outdated version in its URL path, which led to failures when attempting to retrieve item JSON data files. The filename in the latest version is lol.stringtable.json. This update modifies the URL to align with the newest version, ensuring successful data retrieval.

@jjmaldonis
Copy link
Member

Is it necessary to remove the version from the string and always use the latest? That isn't desirable behavior so hopefully it's not necessary.

@ruoyu-lu
Copy link
Author

I just saw that as of version 14.15, the official filename has changed from 'main.stringtable.json' to 'lol.stringtable.json', which will prevent the current project from pulling the file, so I think it's fine to just use the latest version of json

@PushaBe
Copy link

PushaBe commented Oct 5, 2024

isn't it already fixed here anyway? so there is a room to choose version for the plaintext but not for the items themselves as far as I see

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants