[Yadhuthhiyyakat]

[vercel]

@Yadhuthhiyyakat is attempting to deploy a commit to the awindsr's projects Team on Vercel.

A member of the Team first needs to authorize it.

[netlify]

👷 Deploy request for staging-mulearn pending review.

Visit the deploys page to approve it

Name	Link
🔨 Latest commit	9d74b3d

[awindsr]

@greptileai

[greptile-apps]

Confidence Score: 4/5

The PR is not yet safe to merge because bulk selection still submits broadcast notification IDs that the endpoint skips while reporting them as marked read.

Findings

1.  Selection Includes Skipped Broadcasts ▶

Summary

The latest changes add paginated loading to the unified notification feed and flatten fetched pages for display in the notification popover.

• Replaces the single-page notification query with a TanStack infinite query.
• Adds a “Load older notifications” action when more pages are available.
• Keeps feed and unread-count cache invalidation integrated with notification mutations.
• The earlier admin-dispatch, clear-personal, and pagination findings are resolved; selection still includes broadcast notifications even though the bulk-read endpoint skips them.

Diagram

sequenceDiagram
  participant User
  participant Popover as Notification Popover
  participant Query as Infinite Query
  participant API as Notification API
  User->>Popover: Open notifications
  Popover->>Query: Request page 1
  Query->>API: "GET feed?page=1&page_size=20"
  API-->>Query: Results and pagination metadata
  Query-->>Popover: Render accumulated pages
  User->>Popover: Load older notifications
  Popover->>Query: fetchNextPage()
  Query->>API: GET next page
  API-->>Query: Older notifications
  Query-->>Popover: Flatten and render all pages

Loading

_{Reviews (5) · Last reviewed commit: "Notification pagination fixed"}

[greptile-apps]

Selection Includes Skipped Broadcasts

This selection list includes every unread result even though the code states that the bulk-read endpoint skips broadcast IDs. Selecting both personal and broadcast items sends all IDs and reports the full submitted count as marked read, while skipped broadcasts remain unread.

[awindsr]

fix the issues mentioned