Skip to content

Conversation

KoloInDaCrib
Copy link

@KoloInDaCrib KoloInDaCrib commented Aug 24, 2025

What this PR adds?

This PR adds a page for the Script Event Callbacks (i.e. onCreate, onUpdate, etc.), explaining what each does, explaining the different event types and giving some examples for overriding the event callbacks as well as cancelling the events.

Drafted because of Kade's comment regarding accepting community chapters as well as because I have no idea where to place this chapter (hence the numbered index 99).

@Kade-github
Copy link
Member

Kade-github commented Aug 30, 2025

Drafted because of Kade's comment #2 (comment) as well as because I have no idea where to place this chapter (hence the numbered index 99).

Just released the authoring page, feel free to look over it and check what you have rn. If everything is good, feel free to open this pr up and I'll take a look.

EDIT: for your previous concern as well, I think this should be number 8 in the advanced tree (it doesn't really matter since if another one gets published b4 urs, we can just move stuff around to fit)

@KoloInDaCrib KoloInDaCrib marked this pull request as ready for review August 30, 2025 07:10
@Kade-github Kade-github self-requested a review August 30, 2025 22:25
Copy link
Member

@Kade-github Kade-github left a comment

Choose a reason for hiding this comment

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

NAMING: File name should not contain spaces.

Otherwise, looks fantastic!

Copy link
Member

@Kade-github Kade-github left a comment

Choose a reason for hiding this comment

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

goat

@Kade-github Kade-github merged commit 2ec2259 into FunkinCrew:master Aug 31, 2025
@KoloInDaCrib KoloInDaCrib deleted the straight-up-scripting-my-events branch August 31, 2025 09:52
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