feat(ui): clavis delete pca modal, added docs with implemented clavis functionality#837
Conversation
Signed-off-by: Vladislav Schur <u.shchur@sap.com>
Signed-off-by: Vladislav Schur <u.shchur@sap.com>
📝 WalkthroughWalkthroughThis PR implements PCA (Clavis) certificate authority deletion functionality with a confirmation-text modal, renames the create modal for naming consistency, fixes backend service routing from ChangesPCA Delete Modal and Refactoring
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes Possibly related issues
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR adds PCA delete support to the Aurora Portal UI, aligns the delete BFF route with the PCA service key, renames the create modal to PCA terminology, and documents the current Clavis/PCA integration.
Changes:
- Adds a typed-confirmation
DeletePcaModaland wires it into the PCA table row action. - Renames
CreateCaModaltoCreatePcaModaland updates related tests/imports. - Updates PCA delete router behavior/tests, localization catalogs, and adds Clavis/PCA feature documentation.
Reviewed changes
Copilot reviewed 12 out of 14 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
apps/aurora-portal/src/server/Services/routers/pcaRouter.ts |
Switches PCA delete service lookup/validation to pca. |
apps/aurora-portal/src/server/Services/routers/pcaRouter.test.ts |
Updates delete unavailable-service test wording. |
apps/aurora-portal/src/locales/en/messages.ts |
Regenerates English Lingui catalog with delete modal strings. |
apps/aurora-portal/src/locales/en/messages.po |
Adds English delete modal message entries. |
apps/aurora-portal/src/locales/de/messages.ts |
Regenerates German Lingui catalog with new fallback strings. |
apps/aurora-portal/src/locales/de/messages.po |
Adds German catalog entries for new delete modal strings. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/PcaListContainer.tsx |
Updates create modal import/render name and path. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-table/PcaTableRow.tsx |
Enables Delete CA row action and renders delete modal. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-table/PcaTableRow.test.tsx |
Updates row action test for modal opening. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/DeletePcaModal.tsx |
Adds PCA delete confirmation modal. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/DeletePcaModal.test.tsx |
Adds tests for delete confirmation gating and mutation payload. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/CreatePcaModal.tsx |
Renames create modal component/mutation variables to PCA terminology. |
apps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/CreatePcaModal.test.tsx |
Updates create modal test imports and suite name. |
apps/aurora-portal/docs/0011_clavis.md |
Documents current Clavis/PCA UI and BFF scope. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@apps/aurora-portal/src/client/routes/_auth/projects/`$projectId/services/pca/-components/-table/PcaTableRow.tsx:
- Line 42: The render can crash when pca.configuration exists but subject is
undefined; update the cell rendering (the DataGridCell that uses
pca.configuration?.subject.common_name) to guard subject as well and use nullish
coalescing: access pca.configuration?.subject?.common_name ?? "—" so both
missing configuration.subject and empty values are handled safely.
In `@apps/aurora-portal/src/locales/de/messages.po`:
- Around line 634-636: The PO file has empty German translations for several
msgid entries (e.g., msgid "Delete certificate authority" and the other
untranslated msgid groups at the ranges you noted), causing untranslated UI in
the PCA delete flow; update each corresponding msgstr with the proper German
text (for example provide translations for "Delete certificate authority" and
the other msgid strings referenced at 673-675, 679-681, 2515-2517, 2635-2637,
2665-2667) by replacing the empty msgstr values with the correct German phrases,
ensuring plural forms or context markers (if present) are preserved and the PO
file encoding/formatting remains unchanged.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 55eb665a-c418-43fb-b6eb-24352e43fee5
📒 Files selected for processing (14)
apps/aurora-portal/docs/0011_clavis.mdapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/CreatePcaModal.test.tsxapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/CreatePcaModal.tsxapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/DeletePcaModal.test.tsxapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-modals/DeletePcaModal.tsxapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-table/PcaTableRow.test.tsxapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/-table/PcaTableRow.tsxapps/aurora-portal/src/client/routes/_auth/projects/$projectId/services/pca/-components/PcaListContainer.tsxapps/aurora-portal/src/locales/de/messages.poapps/aurora-portal/src/locales/de/messages.tsapps/aurora-portal/src/locales/en/messages.poapps/aurora-portal/src/locales/en/messages.tsapps/aurora-portal/src/server/Services/routers/pcaRouter.test.tsapps/aurora-portal/src/server/Services/routers/pcaRouter.ts
Summary
Added delete-modal with connected delete-bff-endpoint
Changes Made
Related Issues
Checklist