Skip to content

Lily/MoreEvents: added "whenProjectPaused" and "whenProjectUnPause"#2040

Closed
The11thPencil wants to merge 5 commits intoTurboWarp:masterfrom
The11thPencil:pauseEvents
Closed

Lily/MoreEvents: added "whenProjectPaused" and "whenProjectUnPause"#2040
The11thPencil wants to merge 5 commits intoTurboWarp:masterfrom
The11thPencil:pauseEvents

Conversation

@The11thPencil
Copy link
Copy Markdown

No description provided.

@github-actions github-actions Bot added the pr: change existing extension Pull requests that change an existing extension label Mar 18, 2025
@The11thPencil
Copy link
Copy Markdown
Author

!format

const topBlockId = script.blockId;
for (let j = 0; j < runtime.threads.length; j++) {
if (
runtime.threads[j].topBlock === topBlockId &&
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This does not account for clones properly. The script is only going to run in one of the clones at a time since theyll have the same top block ID

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

i am not quite sure how to fix this..

Comment thread extensions/Lily/MoreEvents.js Outdated
Comment thread extensions/Lily/MoreEvents.js Outdated
@yuri-kiss

This comment was marked as abuse.

@The11thPencil
Copy link
Copy Markdown
Author

wait why is this closed, i didnt meant to close it lol

@The11thPencil The11thPencil reopened this Apr 21, 2025
@LilyMakesThings
Copy link
Copy Markdown
Contributor

I'm not a big fan of doing this as it is interfacing with a feature that is introduced through an addon and nothing internally. I'd rather it be introduced into a separate extension like Runtime Options, not here.

@Brackets-Coder
Copy link
Copy Markdown
Contributor

!format

@Brackets-Coder
Copy link
Copy Markdown
Contributor

Brackets-Coder commented Feb 22, 2026

@fath11 this is similar to (though not an exact duplicate of) #1245. According to #2271, we should do our best to avoid similar/duplicate pull requests by encouraging collaboration instead of "one is better than the other". Since @LilyMakesThings would prefer this kind of functionality in a separate extension and #1245 already has this functionality, @fath11 are you okay if I close this? You could always collaborate with @SharkPool-SP to get this extra pause functionality to Turbowarp

@The11thPencil
Copy link
Copy Markdown
Author

Yes I am okay if this is closed

@Brackets-Coder
Copy link
Copy Markdown
Contributor

Okay, hope this won't upset you

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

Labels

pr: change existing extension Pull requests that change an existing extension

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants