Skip to content

Conversation

Copy link

Copilot AI commented Feb 3, 2026

Notification links currently use project IDs (/projects/{id}), but project routes are slug-based (/projects/[slug]), resulting in broken links.

Changes

Type Definition (types/notifications.ts)

  • Added projectSlug?: string to notification data interface

Shared Utility (lib/notifications.ts)

  • Created getProjectUrlFromNotification() to centralize URL generation logic
  • Prefers slug over ID, returns empty string if neither exists

Components Updated

  • NotificationItem.tsx - Link generation uses shared utility
  • NotificationDropdown.tsx - Navigation uses shared utility

Example

// lib/notifications.ts
export const getProjectUrlFromNotification = (
  data: Notification['data']
): string => {
  if (data.projectSlug) {
    return `/projects/${data.projectSlug}`;
  }
  if (data.projectId) {
    return `/projects/${data.projectId}`;
  }
  return '';
};

Backend Requirements

Backend must include projectSlug in notification payloads:

{
  "projectId": "123",
  "projectName": "My Project",
  "projectSlug": "my-project"
}

Until backend sends projectSlug, links will continue using projectId as fallback.

Original prompt

Create an issue in boundlessfi/boundless to handle the links in th notification... instead of using id use slug for projects


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Summary by CodeRabbit

  • Improvements

    • Notification links to projects now use a unified URL resolution mechanism that intelligently identifies projects by slug when available, with automatic fallback to project ID for consistency.
  • Refactor

    • Consolidated project URL construction logic across notification components into a centralized utility function, reducing duplication and improving maintainability.

@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
boundless-kd16 Ready Ready Preview, Comment Feb 3, 2026 8:25pm
boundless-t4ms Ready Ready Preview, Comment Feb 3, 2026 8:25pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
boundless Ignored Ignored Preview Feb 3, 2026 8:25pm

Copilot AI and others added 3 commits February 3, 2026 20:20
Co-authored-by: 0xdevcollins <90073781+0xdevcollins@users.noreply.github.com>
Co-authored-by: 0xdevcollins <90073781+0xdevcollins@users.noreply.github.com>
…or missing fields

Co-authored-by: 0xdevcollins <90073781+0xdevcollins@users.noreply.github.com>
Copilot AI changed the title [WIP] Update notification links to use project slug Use project slugs instead of IDs in notification links Feb 3, 2026
Copilot AI requested a review from 0xdevcollins February 3, 2026 20:25
@0xdevcollins 0xdevcollins marked this pull request as ready for review February 3, 2026 21:14
@coderabbitai
Copy link

coderabbitai bot commented Feb 3, 2026

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • ✅ Full review completed - (🔄 Check again to review again)

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


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

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