Skip to content

feat: reorder API page cards and remove duplicate namespace filter#388

Merged
kaviththiranga merged 1 commit intoopenchoreo:mainfrom
kaviththiranga:api-page-fixes
Mar 12, 2026
Merged

feat: reorder API page cards and remove duplicate namespace filter#388
kaviththiranga merged 1 commit intoopenchoreo:mainfrom
kaviththiranga:api-page-fixes

Conversation

@kaviththiranga
Copy link
Contributor

@kaviththiranga kaviththiranga commented Mar 12, 2026

Move Providers/Consumers cards to top and About/Relations to bottom on
API entity page, remove unused Links card. Remove duplicate
EntityNamespacePicker from API listing page (already in table header).

Before:
Screenshot 2026-03-13 at 03 32 49
Screenshot 2026-03-13 at 03 32 58

After:
Screenshot 2026-03-13 at 03 33 51
Screenshot 2026-03-13 at 03 34 01

Summary by CodeRabbit

  • Removed namespace filtering option from API explorer and mobile filter interface
  • Reorganized API page layout to display component provisioning and consumption information more prominently in the main overview section, improving visibility of component relationships

  Move Providers/Consumers cards to top and About/Relations to bottom on
  API entity page, remove unused Links card. Remove duplicate
  EntityNamespacePicker from API listing page (already in table header).

Signed-off-by: Kavith Lokuhewage <kaviththiranga@gmail.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 12, 2026

📝 Walkthrough

Walkthrough

This pull request removes the EntityNamespacePicker filter component from the CustomApiExplorerPage and reorganizes the card layout in EntityPage's apiPage section, moving EntityProvidingComponentsCard and EntityConsumingComponentsCard into the top overview area while removing the EntityLinksCard from that section.

Changes

Cohort / File(s) Summary
Catalog API Explorer & Entity Page UI
packages/app/src/components/catalog/CustomApiExplorerPage.tsx, packages/app/src/components/catalog/EntityPage.tsx
Removed EntityNamespacePicker filter component and reorganized card layout in apiPage section, promoting component provisioning/consumption cards to the top row and removing the links card from this region.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • sameerajayasoma
  • jhivandb
  • stefinie123

Poem

🐰 Away with the namespace picker we go,
Cards shuffled 'round in a cleaner flow,
Providing and consuming take center stage,
The catalog page turns a bright new page! ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description provides a clear summary with before/after screenshots but lacks several required template sections including Purpose, Goals, Approach, User stories, Release notes, and other documentation/testing details. Complete the PR description by adding Purpose, Goals, Approach, Release note, Documentation, and relevant sections from the template to provide comprehensive context for reviewers.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes both main changes: reordering API page cards and removing the duplicate namespace filter component.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

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

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🧹 Nitpick comments (1)
packages/app/src/components/catalog/EntityPage.tsx (1)

487-494: Missing spacing and xs props may cause layout inconsistencies.

The nested Grid container lacks spacing={3} (causing no gap between the two cards), and the inner items lack xs={12} for mobile responsiveness. Other card pairs in this file (e.g., lines 495-504) follow a consistent pattern.

♻️ Suggested fix for consistent layout and responsiveness
-        <Grid container item md={12}>
-          <Grid item md={6}>
+        <Grid container item md={12} spacing={3}>
+          <Grid item md={6} xs={12}>
             <EntityProvidingComponentsCard />
           </Grid>
-          <Grid item md={6}>
+          <Grid item md={6} xs={12}>
             <EntityConsumingComponentsCard />
           </Grid>
         </Grid>
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@packages/app/src/components/catalog/EntityPage.tsx` around lines 487 - 494,
The nested Grid container wrapping EntityProvidingComponentsCard and
EntityConsumingComponentsCard is missing spacing and the inner Grid items are
missing responsive xs props; update the parent Grid (the container with md={12})
to include spacing={3} and add xs={12} to each inner Grid item that wraps
EntityProvidingComponentsCard and EntityConsumingComponentsCard to match the
other card pairs and ensure mobile responsiveness and consistent gaps.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@packages/app/src/components/catalog/EntityPage.tsx`:
- Around line 487-494: The nested Grid container wrapping
EntityProvidingComponentsCard and EntityConsumingComponentsCard is missing
spacing and the inner Grid items are missing responsive xs props; update the
parent Grid (the container with md={12}) to include spacing={3} and add xs={12}
to each inner Grid item that wraps EntityProvidingComponentsCard and
EntityConsumingComponentsCard to match the other card pairs and ensure mobile
responsiveness and consistent gaps.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 5d015203-4f66-49ac-86b1-b675431b4ad2

📥 Commits

Reviewing files that changed from the base of the PR and between 009d5c9 and 9e5d67f.

📒 Files selected for processing (2)
  • packages/app/src/components/catalog/CustomApiExplorerPage.tsx
  • packages/app/src/components/catalog/EntityPage.tsx
💤 Files with no reviewable changes (1)
  • packages/app/src/components/catalog/CustomApiExplorerPage.tsx

@kaviththiranga kaviththiranga merged commit 87d7f40 into openchoreo:main Mar 12, 2026
3 checks passed
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.

2 participants