[API Portal] Display key manager name instead of uuid

[Piumal1999]

Purpose

$subject

Summary

Previously the key manager UI was showing the id of keymanager instead of the display name. Fixed the issue with this PR.

Samples

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: f998c962-9da8-4469-b7fc-2c6eadc4957e

📥 Commits

Reviewing files that changed from the base of the PR and between d34d96f and da437f4.

📒 Files selected for processing (4)

• portals/api-portal/it/ui/cypress/e2e/applications/application-flows.cy.js
• portals/api-portal/src/controllers/applicationsContentController.js
• portals/api-portal/src/pages/application/partials/manage-keys.hbs
• portals/api-portal/src/scripts/oauth2-key-generation.js

---

📝 Walkthrough

Walkthrough

The application portal now includes key manager display names in metadata and renders them in production and sandbox cards. OAuth2 key-generation requests use the key manager ID. The Cypress flow verifies the display name.

Changes

Key manager display-name flow

Layer / File(s)	Summary
Metadata and card wiring portals/api-portal/src/controllers/applicationsContentController.js, portals/api-portal/src/pages/application/partials/manage-keys.hbs	Key manager metadata now includes name. Production and sandbox cards use name for display and retain id as kmId.
OAuth2 request and UI validation portals/api-portal/src/scripts/oauth2-key-generation.js, portals/api-portal/it/ui/cypress/e2e/applications/application-flows.cy.js	The key-generation payload uses kmId as keyManager. The Cypress flow seeds and verifies the key manager display name.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

• wso2/api-platform#2900: Related Applications key-manager E2E coverage and display-name behavior.

Suggested reviewers: lasanthas, virajsalaka, thushani-jayasekera

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description states the issue and includes a screenshot, but it omits most required sections and leaves the Purpose placeholder unchanged.	Complete the required Purpose, Goals, Approach, User stories, Documentation, Automation tests, Security checks, Related PRs, and Test environment sections.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly identifies the main change: displaying the key manager name instead of its UUID.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}