Skip to content

fix(admin): show owner identity in ownership transfers - #1720

Merged
ctkm-aelf merged 1 commit into
mainfrom
ownership-transfer-user-names
Oct 1, 2026
Merged

ctkm-aelf merged 1 commit into
mainfrom
ownership-transfer-user-names

Conversation

@ctkm-aelf

Copy link
Copy Markdown
Collaborator

Summary

  • Add owner_name and owner_email to the admin ownership inventory response using one batched users lookup for the current page.
  • Show the owner's name, email, and UUID in the transfer inventory. Missing names fall back to email, and missing accounts retain the UUID.
  • Keep the frontend on its existing single inventory HTTP request; no per-owner API requests are made.

Verification

  • npm test -- --run src/pages/admin-ownership.test.tsx src/components/shared/ownership-transfer-dialog.test.tsx (7 passed)
  • npx tsc -b --pretty false
  • Targeted ESLint and Prettier checks
  • cargo fmt --check --all
  • cargo check -p nyxid --tests

The MongoDB integration test was compiled but not executed because this environment has no configured replica-set test database.

@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

📊 Code coverage

Component Lines Threshold Status Δ vs base
Backend (nyxid) 87.69% 73% ✅ — 0.00
Frontend (vitest) 71.45% 15% ✅ 🔻 -0.01

Gate: line coverage must stay at or above the threshold. Ratchet plan (W21): Backend → 55%, CLI → 50%, Frontend → 30% by quarter end.

@ctkm-aelf
ctkm-aelf merged commit 80a153d into main Oct 1, 2026
34 checks passed
@ctkm-aelf
ctkm-aelf deleted the ownership-transfer-user-names branch October 1, 2026 04:05
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.

1 participant