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

Remove visual scripting from the main documentation #6111

Merged
merged 1 commit into from
Aug 26, 2022

Conversation

skyace65
Copy link
Contributor

@skyace65 skyace65 commented Aug 26, 2022

Removes visual scripting from the documentation since its been removed from the engine(godotengine/godot#64822). I've deleted the entire section and references I could find to it by searching for "visualscript" in the docs. I also removed it from the redirects page.

@mhilbrunner mhilbrunner added the content:redirect Issues and PRs involving moving content and adding a redirect rule on the backend label Aug 26, 2022
@mhilbrunner
Copy link
Member

mhilbrunner commented Aug 26, 2022

Built and tested locally, grepping for 'Visual Script', 'VisualScript' and 'Visual*Script' didn't result in anything missed as far as I can see. The one edited place that mentions GDNative needs to be updated for GDExtension, but that can be done in a PR that does that for the whole docs. Good work!

@mhilbrunner mhilbrunner merged commit 19333e5 into godotengine:master Aug 26, 2022
wirecat added a commit to wirecat/godot-docs that referenced this pull request Mar 5, 2023
Grammar issue accidentally left in when references to Visual Script
were removed. godotengine#6111
wirecat added a commit to wirecat/godot-docs that referenced this pull request Mar 6, 2023
Grammar issue accidentally left in when references to Visual Script
were removed. godotengine#6111
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug content:redirect Issues and PRs involving moving content and adding a redirect rule on the backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants