Skip to content

Compatibility with Godot 4.7 - #84

Open
DarkGrey456 wants to merge 20 commits into
dreadpon:masterfrom
DarkGrey456:master
Open

Compatibility with Godot 4.7#84
DarkGrey456 wants to merge 20 commits into
dreadpon:masterfrom
DarkGrey456:master

Conversation

@DarkGrey456

Copy link
Copy Markdown

Got the addon to work in Godot 4.7, made the return type explicit from a couple of functions and returned default values where needed,

Comment thread addons/dreadpon.spatial_gardener/plugin.gd Outdated
Comment thread addons/dreadpon.spatial_gardener/arborist/arborist.gd Outdated
Comment thread addons/dreadpon.spatial_gardener/gardener/gardener.gd
Comment thread addons/dreadpon.spatial_gardener/gardener/gardener.gd
DarkGrey456 and others added 5 commits July 6, 2026 13:47
Co-authored-by: rphcos <91429946+RphCos@users.noreply.github.com>
Co-authored-by: rphcos <91429946+RphCos@users.noreply.github.com>
Co-authored-by: rphcos <91429946+RphCos@users.noreply.github.com>
Co-authored-by: rphcos <91429946+RphCos@users.noreply.github.com>
…input_field_resource.gd

Co-authored-by: rphcos <91429946+RphCos@users.noreply.github.com>
@DarkGrey456

Copy link
Copy Markdown
Author

I applied the changes, thanks, I didnt really understand some of that. The addon seemed to be working but I did notice some problems after I made the PR which was uploaded using the webUI, so I will test this with the changes and update my results in the comments.

@RphCos

RphCos commented Jul 6, 2026

Copy link
Copy Markdown

@DarkGrey456 Some of the issue I flagged only happened when you opened an existing Garderner, and one only appeared once on project start, and none seem to be blocking/breaking

Thank you for doing most of the heavy work!

@DarkGrey456

Copy link
Copy Markdown
Author

I had to revert/update the Gardener.gd file again to get this working

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.

2 participants