Skip to content

[Improvement]: Add inactive-only option to Classification Store delete command#18765

Merged
kingjia90 merged 2 commits into12.xfrom
delete-cskey-command
Oct 23, 2025
Merged

[Improvement]: Add inactive-only option to Classification Store delete command#18765
kingjia90 merged 2 commits into12.xfrom
delete-cskey-command

Conversation

@kingjia90
Copy link
Copy Markdown
Contributor

@kingjia90 kingjia90 commented Oct 14, 2025

Changes in this pull request

Resolves pimcore/admin-ui-classic-bundle#493

Additional info

Not entirely sure if it's worth to have it in one command or should be split into a different name

@github-actions
Copy link
Copy Markdown

Review Checklist

  • Target branch (11.5 for bug fixes, others 12.x)
  • Tests (if it's testable code, there should be a test for it - get help)
  • Docs (every functionality needs to be documented, see here)
  • Migration incl. install.sql (e.g. if the database schema changes, ...)
  • Upgrade notes (deprecations, important information, migration hints, ...)
  • Label
  • Milestone

@kingjia90 kingjia90 force-pushed the delete-cskey-command branch from 84d49ff to c35868d Compare October 14, 2025 15:27
@kingjia90 kingjia90 self-assigned this Oct 23, 2025
@sonarqubecloud
Copy link
Copy Markdown

@kingjia90 kingjia90 merged commit 2eb56ef into 12.x Oct 23, 2025
20 checks passed
@kingjia90 kingjia90 deleted the delete-cskey-command branch October 23, 2025 15:54
@github-actions github-actions bot locked and limited conversation to collaborators Oct 23, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Classification Store: Keys are not properly deleted

2 participants