Skip to content

fix(refs): resolve logical group statuses - #161

Draft
bwl21 wants to merge 9 commits into
eqrm:mainfrom
bwl21:fix/157-logical-group-statuses
Draft

bwl21 wants to merge 9 commits into
eqrm:mainfrom
bwl21:fix/157-logical-group-statuses

Conversation

@bwl21

@bwl21 bwl21 commented Aug 29, 2026 •

Copy link
Copy Markdown
Contributor

Merge order and gate

This is step 2 of the intended issue order: #143 → #157 → #164.

The source branches live in the bwl21/ct-cli fork. GitHub does not support cross-fork stacked PR bases, so this PR technically targets main; the draft state is the explicit merge-order gate.

Summary

  • load the group-status catalog from /person/masterdata and resolve technical status names to host-local IDs
  • reverse-resolve known groupStatusId values during adoption while preserving unknown numeric IDs with an actionable portability warning
  • keep existing numeric input backward compatible and distinguish group statuses from membership and person statuses in documentation and code comments
  • cover hosts that assign different numeric IDs to the same technical status name

Testing

  • npm run test — 1,083 passed, 5 skipped
  • npm run typecheck
  • npm run lint
  • npm run format:check
  • npm run build
  • node .github/scripts/docs-staleness.mjs
  • mkdocs build -f docs/handbuch/mkdocs.yml --strict

Closes #157

@bwl21
bwl21 force-pushed the fix/157-logical-group-statuses branch from e11c3eb to b8be1ef Compare September 4, 2026 09:27
@bwl21
bwl21 marked this pull request as draft September 4, 2026 09:34
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.

bug(refs): resolve group statuses from /person/masterdata instead of treating #67 as permanent

1 participant