Skip to content

Conversation

@MannyC
Copy link
Contributor

@MannyC MannyC commented Oct 9, 2023

Godot 4.0 added a new parameter to EditorInspectorPlugin._parse_property. This was updated in the C# version of the tutorial, but not the gdscript version

@AThousandShips
Copy link
Member

You will need to update the use of path below

@AThousandShips AThousandShips added bug topic:gdscript area:manual Issues and PRs related to the Manual/Tutorials section of the documentation cherrypick:4.0 cherrypick:4.1 labels Oct 11, 2023
@MannyC
Copy link
Contributor Author

MannyC commented Oct 11, 2023

You will need to update the use of path below

Yeah, I had seen that but I couldn't see how to include it in a batch commit using the on-site UI.

Do you want these squashed of does it not matter?

Copy link
Member

@AThousandShips AThousandShips left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Please squash your commits into one, see here

@MannyC
Copy link
Contributor Author

MannyC commented Oct 11, 2023

Hmm, give me another second on this one

Godot 4.0 added a new parameter to EditorInspectorPlugin._parse_property.  This was updated in the C# version of the tutorial, but not the gdscript version

Co-authored-by: A Thousand Ships <96648715+AThousandShips@users.noreply.github.com>
@MannyC
Copy link
Contributor Author

MannyC commented Oct 11, 2023

Okay, should be good to go

@mhilbrunner mhilbrunner merged commit 971c1f7 into godotengine:master Oct 13, 2023
@mhilbrunner
Copy link
Member

Merged. Thanks and congrats on your first merged contribution!

@MannyC MannyC deleted the patch-1 branch October 14, 2023 11:17
@mhilbrunner
Copy link
Member

Cherry-picked to 4.1.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:manual Issues and PRs related to the Manual/Tutorials section of the documentation bug cherrypick:4.0 topic:gdscript

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants