Change Application page UI

[Induwara04]

This pull request simplifies the ApplicationOverview page by removing the toggle between "Associations" and "API Keys" views, leaving only the "Associations" tab visible. Unused imports and state related to the removed functionality have also been cleaned up.

UI simplification:

• Removed the ToggleButtonGroup and related state (viewMode) that allowed switching between "Associations" and "API Keys" tabs, so only the "Associations" tab is now shown. [1] [2]

Code cleanup:

• Removed imports for ToggleButton, ToggleButtonGroup, and the APIKeyTab component, as they are no longer used. [1] [2]

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: e3a53115-cda3-4511-8036-4bec5cc86d8d

📥 Commits

Reviewing files that changed from the base of the PR and between 8e7f47c and db6f728.

📒 Files selected for processing (1)

• portals/ai-workspace/src/pages/appShell/appShellPages/applications/ApplicationOverview.tsx

---

📝 Walkthrough

Updated the application overview page to always show the “Associated GenAI Resources” associations view.

• Removed the toggle UI for switching between “Associations” and “API Keys”
• Dropped now-unused state and imports tied to the removed view switcher
• Kept the associations content as the only rendered view when an application is selected

Walkthrough

ApplicationOverview.tsx is simplified by removing the toggle between "Associated GenAI Resources" and "API Keys" views. The ToggleButton and ToggleButtonGroup oxygen-ui imports are removed, along with the APIKeyTab import and the viewMode React state. The conditional rendering block that previously switched between associations and API keys based on viewMode is replaced with a single path that renders ApplicationAssociationsProvider and AssociationsTab unconditionally when applicationId is available.

🚥 Pre-merge checks | ✅ 2 | ❌ 3

❌ Failed checks (2 warnings, 1 inconclusive)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description captures the change summary, but it misses most required template sections like Purpose, Goals, tests, and security checks.	Add the missing template sections: Purpose, Goals, Approach, User stories, Documentation, tests, security checks, samples, related PRs, and test environment.
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.
Title check	❓ Inconclusive	The title is generic and doesn't clearly identify the main change in the ApplicationOverview page.	Rename it to reflect the UI simplification, e.g. 'Remove API Keys toggle from ApplicationOverview'.

✅ Passed checks (2 passed)

Check name	Status	Explanation
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.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ 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.}