Skip to content

Fix string script commands being broken by update - #2137

Open
ccuser44 wants to merge 3 commits into
Epix-Incorporated:masterfrom
ccuser44:patch-54
Open

Fix string script commands being broken by update#2137
ccuser44 wants to merge 3 commits into
Epix-Incorporated:masterfrom
ccuser44:patch-54

Conversation

@ccuser44

@ccuser44 ccuser44 commented May 17, 2026

Copy link
Copy Markdown
Contributor

ccuser44 added 2 commits May 17, 2026 02:53
Added checks to notify players when code execution is disabled for certain commands.
@ccuser44
ccuser44 marked this pull request as draft May 17, 2026 11:18
@ccuser44 ccuser44 changed the title Better UX for certain commands Fix string script commands being broken by update May 17, 2026
@ostrichman1

Copy link
Copy Markdown
Contributor

I get that my fix to try and rectify this probably wasn't the best but is there not another solution other than just compeltely disabling the command?

@ccuser44

Copy link
Copy Markdown
Contributor Author

I get that my fix to try and rectify this probably wasn't the best but is there not another solution other than just compeltely disabling the command?

That's exactly what this pull request doesn't do

@ccuser44
ccuser44 marked this pull request as ready for review June 14, 2026 12:21
@Dimenpsyonal

Copy link
Copy Markdown
Member

I'm confused why :tornado specifically needs code execution to work

@Dimenpsyonal Dimenpsyonal added ⚡ command Specific Adonis commands ⌛needs PoF This PR is awaiting proof of functionality labels Aug 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

⚡ command Specific Adonis commands ⌛needs PoF This PR is awaiting proof of functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants