Skip to content

[OpenSpec] integration-occ-set-theme #681

Description

@github-actions

⚠️ OpenSpec-managed issue — this content is automatically synced
from the openspec/ directory. Manual edits will be overwritten on next sync.

Artifacts

Specs

Tasks

  • 1.1 Move the body of SettingsController::setTokenSet() into TokenSetService::activate($id, $actor); the controller calls it. Verify: SettingsControllerTest stays green and a new TokenSetServiceTest case covers invalid id, unchanged id (no audit entry) and a switch.
  • 2.1 nldesign:theme:list and nldesign:theme:get. Verify: tests/Unit/Command/ThemeListTest.php and ThemeGetTest.php assert the table columns and the group mapping output.
  • 2.2 nldesign:theme:set <token-set> [--sync-core] [--dry-run], registered in appinfo/info.xml. Verify: tests/Unit/Command/ThemeSetTest.php for success, unknown id (non-zero exit), --dry-run (no write) and --sync-core (calls ThemingService).
  • 3.1 Docs: the three commands in docs/ with a scripting example. Verify: the docs build.
  • 3.2 Switch to an incomplete custom set from the command line and check the page falls back to defaults. Verify: manual check recorded in the PR.

Design

See design.md for technical design details.


Synced from openspec/changes/integration-occ-set-theme by OpenSpec workflow
App: thematiq

Activity

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions