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

Check if the class name for /class exists #9

Open
winston-yallow opened this issue Dec 13, 2023 · 2 comments
Open

Check if the class name for /class exists #9

winston-yallow opened this issue Dec 13, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@winston-yallow
Copy link
Owner

This should make a web request to the docs and see if it returns code 404. In that case an ephemeral warning should be shown, otherwise the link with a short summary.

This issue should only be implemented after #8 is done to prevent sending too many web requests. Alternatively, results should be cached for some time.

@winston-yallow winston-yallow added the enhancement New feature or request label Dec 13, 2023
@winston-yallow
Copy link
Owner Author

This can also include fuzzy matching. For that we would need to update the list of available classes from time to time (maybe a few times per day). That way users can enter "node 2d" to mean "Node2D".

@Xananax
Copy link
Collaborator

Xananax commented Dec 15, 2023

The list of classes is on the Godot repo, so we could just take it from there. In the future, we could ask Godot to have a trigger/webhook that rebuilds the bot.

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

No branches or pull requests

2 participants