Skip to content

Feat: add taxonomy and term publish, unpublish, localize, and unlocalize support - #262

Draft
reeshika-h wants to merge 1 commit into
developmentfrom
feat/DX-9676-taxonomy-publish
Draft

Feat: add taxonomy and term publish, unpublish, localize, and unlocalize support#262
reeshika-h wants to merge 1 commit into
developmentfrom
feat/DX-9676-taxonomy-publish

Conversation

@reeshika-h

Copy link
Copy Markdown
Contributor

Summary

  • Adds Taxonomy.publish() / Taxonomy.unpublish() (POST /taxonomies/publish / /unpublish), closing the parity gap with the JS SDK per DX-9676
  • Adds Taxonomy.localize() / Taxonomy.unlocalize() and Terms.localize() / Terms.unlocalize() for locale-scoped taxonomy/term operations
  • branch header passthrough works via the existing generic addHeader() mechanism, no new parameter needed
  • All six methods were live-verified end-to-end against a real stack (not just request-shape assertions) — publish/unpublish return real job_ids, localize/unlocalize round-trip cleanly (201 Created / 204 No Content) with zero residual changes to real data

Test plan

  • mvn clean testUnitTestSuite: 245 tests, 0 failures; APISanityTestSuite: 219 tests, 0 failures
  • Live-verified publish/unpublish against a real taxonomy (200, real job_id)
  • Live-verified localize/unlocalize at both taxonomy and term level (create → 201, delete → 204, restore confirmed via fetch)
  • Fixed a pre-existing gap where TaxonomyTest.java (package-private) was never wired into UnitTestSuite and so never ran under mvn test

Out of scope

  • Term-level localize/unlocalize was added per follow-up request but is not yet covered in the original DX-9674 TRD text — flagging for TRD update if needed
  • Python/.NET/JS parity fixes from the TRD are not part of this PR (Java-only)

🤖 Generated with Claude Code

…ize support
Closes the parity gap with the JS SDK per DX-9676 by adding CMA taxonomy
publish/unpublish and taxonomy/term-level localize/unlocalize methods,
each live-verified against a real stack.
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@reeshika-h
reeshika-h requested a review from a team as a code ownerJuly 28, 2026 06:09
@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check TypeCount (with fixes)Without fixesThresholdResult
🔴 Critical Severity0010✅ Passed
🟠 High Severity0025✅ Passed
🟡 Medium Severity00500✅ Passed
🔵 Low Severity001000✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

SeverityBreaches (with fixes)Breaches (no fixes)SLA Threshold (with/no fixes)Status
🔴 Critical0015 / 30 days✅ Passed
🟠 High0030 / 120 days✅ Passed
🟡 Medium0090 / 365 days✅ Passed
🔵 Low00180 / 365 days✅ Passed

✅ BUILD PASSED - All security checks passed

OMpawar-21
OMpawar-21 previously approved these changes Jul 29, 2026

@OMpawar-21OMpawar-21 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

aniket-shikhare-cstk
aniket-shikhare-cstk previously approved these changes Aug 2, 2026
@snyk-io

snyk-ioBot commented Aug 7, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

StatusScan Engine Critical High Medium LowTotal (0)
Open Source Security0000 0 issues
Licenses0000 0 issues
Code Security0000 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@OMpawar-21
OMpawar-21 dismissed stale reviews from aniket-shikhare-cstk and themself via e31021dAugust 11, 2026 13:58
@OMpawar-21
OMpawar-21force-pushed the feat/DX-9676-taxonomy-publish branch from 1a9a97b to e31021dCompareAugust 11, 2026 13:58
@reeshika-h
reeshika-h marked this pull request as draft August 18, 2026 06:39
Sign up for freeto 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

@reeshika-h@OMpawar-21@aniket-shikhare-cstk