-
Notifications
You must be signed in to change notification settings - Fork 4.2k
feat(dashboard,api): context activity feed, search & filtering fixes NV-6758 #9285
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(dashboard,api): context activity feed, search & filtering fixes NV-6758 #9285
Conversation
✅ Deploy preview added
To edit notification comments on pull requests, go to your Netlify project configuration. |
@coderabbitai summary |
✅ Actions performedSummary regeneration triggered. |
WalkthroughAdds optional context-based search to activity feeds end-to-end: DTOs, controller with feature flag gating, command/use case propagation, repository query and schema index, dashboard API/types/hooks, and UI components (filters, drawers, lists) updated to read/write a new contextSearch field and render context-driven interactions. Changes
Sequence Diagram(s)sequenceDiagram
autonumber
participant UI as Dashboard UI (Activity/Contexts)
participant FF as Feature Flag (client)
participant API as NotificationsController
participant SFF as Feature Flag (server)
participant UC as GetActivityFeed Usecase
participant Repo as NotificationRepository
participant DB as DB (notifications)
UI->>FF: Check IS_CONTEXT_ENABLED
alt Flag enabled
UI->>API: GET /notifications?…&contextSearch={q}
else Flag disabled
UI->>API: GET /notifications?… (no contextSearch)
end
API->>SFF: Read IS_CONTEXT_ENABLED
alt Server flag enabled
API->>UC: Execute(GetActivityFeedCommand{ contextSearch })
else Disabled
API->>UC: Execute(GetActivityFeedCommand{ contextSearch: undefined })
end
UC->>Repo: getFeed({ …, contextSearch })
alt contextSearch contains ":"
Repo->>DB: Find contextKeys IN [q] with env + pagination
else partial search
Repo->>DB: Find contextKeys regex /q/i with env + pagination
end
DB-->>Repo: Feed items
Repo-->>UC: Items
UC-->>API: ActivitiesResponse
API-->>UI: Rendered feed
Estimated code review effort🎯 4 (Complex) | ⏱️ ~60 minutes Possibly related PRs
Suggested reviewers
Poem
✨ Finishing touches
🧪 Generate unit tests
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (18)
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. Comment |
What changed? Why was the change needed?
Screenshots
Expand for optional sections
Related enterprise PR
Special notes for your reviewer
Summary by CodeRabbit