Skip to content

Conversation

@satrun77
Copy link
Contributor

No description provided.

@satrun77 satrun77 force-pushed the pulls/cms6-support branch 5 times, most recently from 293d801 to dff570f Compare April 29, 2025 04:22
@satrun77 satrun77 force-pushed the pulls/cms6-support branch from dff570f to 1252214 Compare April 29, 2025 04:23
@satrun77 satrun77 marked this pull request as ready for review April 29, 2025 04:29
@wilr wilr merged commit e6cbcbe into dnadesign:main Apr 29, 2025
6 checks passed
@lerni
Copy link
Contributor

lerni commented Aug 28, 2025

@satrun77 It seems colons are not allowed in $commandName. This results in an error like:
[Emergency] Uncaught LogicException: commandName must not contain ':' or '/'. Got elemental:virtual:reset-globally-enabled

Should hyphens be used instead, like elemental-virtual-reset-globally-enabled?
This is preventing the task from being accessible via /dev/tasks.

This appears to have been fixed in the SilverStripe Ltd fork: silverstripeltd@bf0fd5f#diff-fd309435aedc66dd7b565a21d99c891d332752811d0dd15afa00f939107cd322R13

Will you push that forward or should I PR?

Thanks for all your SS6 PRs ❤️

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.

3 participants