H-6625: Improve loading, org invites, action bell routing#8917
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
PR SummaryMedium Risk Overview Signup / org invites: Entities table: “Show more” is driven by an explicit Inbox navigation: New Reviewed by Cursor Bugbot for commit cad4563. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Pull request overview
Frontend-focused improvements across navigation, onboarding, and the supply-chain experience. Key themes are making loading states feel more “app-like”, routing notification entrypoints to the most relevant inbox subsection, and improving reliability of incremental table loading.
Changes:
- Introduced
getInboxHref()and routed both the sidebar “Inbox” link and the header notification bell to the most relevant inbox subsection (actions/notifications/invites). - Replaced several supply-chain “Loading…” placeholder states with a consistent skeleton (
SupplyChainAppSkeleton) and improved scope-selection behavior when data is missing. - Updated signup to auto-handle pending org invitations post-verification (and adjusted API middleware to allow fetching pending invites for logged-in users who haven’t completed signup), plus made “load more” handling more reliable in the entities table.
Reviewed changes
Copilot reviewed 15 out of 15 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| apps/hash-frontend/src/shared/layout/layout-with-sidebar/sidebar.tsx | Uses getInboxHref() so the Inbox sidebar item routes to the most relevant subsection. |
| apps/hash-frontend/src/shared/layout/layout-with-header/notifications-dropdown.tsx | Routes the bell icon to the most relevant inbox target via getInboxHref(). |
| apps/hash-frontend/src/shared/get-inbox-href.ts | Adds a shared helper to compute the best Inbox destination. |
| apps/hash-frontend/src/pages/supply-chain/supply-chain-data-shell/site.tsx | Replaces per-view loading state with the supply-chain skeleton. |
| apps/hash-frontend/src/pages/supply-chain/supply-chain-data-shell/opportunity.tsx | Replaces brief loading UI with the supply-chain skeleton. |
| apps/hash-frontend/src/pages/supply-chain/supply-chain-data-shell.tsx | Adds multi-scope registry loading fallback behavior and uses skeleton + updated empty/error UX. |
| apps/hash-frontend/src/pages/supply-chain/site/[site-id]/opportunity/[opportunity-type]/[product-id]/[step-id].page.tsx | Uses skeleton while router params are not ready. |
| apps/hash-frontend/src/pages/supply-chain/shared/supply-chain-layout.tsx | Uses skeleton while workspace scope is resolving. |
| apps/hash-frontend/src/pages/supply-chain/shared/load-state.tsx | Introduces SupplyChainAppSkeleton to match supply-chain page chrome. |
| apps/hash-frontend/src/pages/supply-chain/product/[product-id].page.tsx | Replaces product-page loading UI with the skeleton. |
| apps/hash-frontend/src/pages/signup.page.tsx | Adds auto-handling for pending invitations after verification and refreshes invites/workspace on accept. |
| apps/hash-frontend/src/pages/shared/entities-visualizer/entities-table.tsx | Makes “has more rows” explicit via prop and improves remaining-count rendering when totals are unknown. |
| apps/hash-frontend/src/pages/shared/entities-visualizer.tsx | Supplies hasMoreRowsAvailable based on cursor presence. |
| apps/hash-frontend/src/pages/_app.page.tsx | Removes supply-chain hidden-pathname gating for the supplyChain feature flag. |
| apps/hash-api/src/graphql/resolvers/index.ts | Allows getMyPendingInvitations for logged-in users who may not have completed signup. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit cad4563. Configure here.

🌟 What is the purpose of this PR?
A few improvements to the frontend: