Skip to content

Conversation

@ueco-jb
Copy link

@ueco-jb ueco-jb commented Nov 7, 2021

closes #5373

It changes line in:
getting_started/step_by_step/files/instanting.zip/Main.gd

- export (PackedScene) var Ball
+ @export var Ball: PackedScene

@Calinou
Copy link
Member

Calinou commented Nov 7, 2021

I think we should remove this ZIP file and point to the demo project in https://github.com/godotengine/godot-demo-projects instead. This would avoid storing ZIP files within the documentation, which should be avoided for various reasons.

@ueco-jb
Copy link
Author

ueco-jb commented Nov 7, 2021

@Calinou I agree in general.
Although in godot-demo-projects there is a similar instacing project: Instancing Demo
Should:

  • this step by step instansing page reflect (in code snippets, for example) already existing example,
  • Instancing Demo be replaced by content of this instanting.zip
  • new directory in godot-demo-projects be created, which would point to all .zip files from getting_started?

I'd prefer 3rd option, because first option is kind of more complicated then instancing.zip, and it'd be a shame to just remove Instancing Demo as well.

@ueco-jb
Copy link
Author

ueco-jb commented Nov 12, 2021

@Calinou ?

@Calinou
Copy link
Member

Calinou commented Nov 12, 2021

@ueco-jb Since demos haven't been updated for Godot 4.0 yet, I would prefer we wait for 4.0 to be more stable to start upgrading all demos to Godot 4.0. Then we can look at updating the current instancing tutorial to reflect the demo's contents.

4.0.dev is not production-ready and therefore not recommended for beginners, so it's not a critical issue if beginner tutorials aren't updated yet. It will be done in due time 🙂

@skyace65 skyace65 added the area:getting started Issues and PRs related to the Getting Started section of the documentation label Jan 13, 2023
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 1, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:

https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 1, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:

https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 1, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:

https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 1, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:

https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 1, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:

https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 1, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:
https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Added a small section about instancing via code.

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 4, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:
https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Added a small section about instancing via code.

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 4, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:
https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Added a small section about instancing via code.

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
rm-code added a commit to rm-code/godot-docs that referenced this pull request Mar 6, 2023
Removed the instancing.zip as discussed in godotengine#6793 and replaced it with
text and image references to the godot-projects instancing demo:
https://github.com/godotengine/godot-demo-projects/tree/4.0/2d/instancing

Added a small section about instancing via code.

Closes godotengine#6793

Supersedes godotengine#6187 and godotengine#5374
@skyace65
Copy link
Contributor

Superseded by #7222

@skyace65 skyace65 closed this Apr 28, 2023
@ueco-jb ueco-jb deleted the 5373-step-by-step-instancing-zip-example-outdated branch June 19, 2023 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

archived area:getting started Issues and PRs related to the Getting Started section of the documentation enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4.0 - Step by step instancing example error in zip

3 participants