Repository navigation
Script overte - #71
Merged
Merged
Conversation
* De-emphasise Overte being just a VR platform * Don't use var in example scripts
* Convert from markdown to reStructuredText format * Fix syntax error in example code * Expand Equip Item section with full working example * Add "use strict" and remove vars in example code
* console.log in an exported function will not show in logs
* "use strict", etc. * Also add a way for desktop users to try out the avatar script by pressing "x" on their keyboard.
* Remove section on using Package Model to add scripts; does not work * Replace image of text editor with code block, including highlighted line
* Encourage use of server entity script in some situations * Correct steps to add a client entity script, with scripts page icon. * Example script now wil "use strict", etc.
* Correct steps to attach a Server Entity Script to an Entity * Example script will now "use strict", etc.
* I don't mean to alarm anyone, but it is now closer to 40 years since the 80s. A slight change to the wording to make this tutorial timeless.
* Loading a script from the UI will load it persistently
* clickDownOnEntity only responds to mouse click events; cannot be used in VR with controllers * mousePressOnEntity responds to both mouse click and controller trigger events
zedwick
marked this pull request as ready for review
February 24, 2026 12:58
JulianGro
requested changes
Feb 27, 2026
Member
|
You can apply multiple suggestions at once from the Files changed page. |
Co-authored-by: Julian Groß <julian.g@posteo.de>
…rototype function.
Co-authored-by: Julian Groß <julian.g@posteo.de>
minor correction Co-authored-by: Julian Groß <julian.g@posteo.de>
JulianGro
requested changes
May 5, 2026
JulianGro
left a comment
Member
There was a problem hiding this comment.
The code blocks still have unintended whitespaces. There is also still a bunch of small suggestions to apply.
GitHub is really buggy for me right now when it comes to code review. Many comments aren't shown on the "Files changed" page and some suggestions aren't rendered correctly. Just a heads up about that.
Contributor
Author
|
I can't even respond to your latest comments directly at the moment, but I've addressed everything I can see. |
Co-authored-by: Julian Groß <julian.g@posteo.de>
JulianGro
approved these changes
May 7, 2026
JulianGro
left a comment
Member
There was a problem hiding this comment.
I took the liberty of applying some of my previous suggestions. I assume you missed them because of the GitHub review mess.
Contributor
Author
|
Looks good, thanks Julian. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR brings the Script section of the docs up to date.