Skip to content

allow pinning dashboards#260

Merged
vkarpov15 merged 3 commits into
mainfrom
IslandRhythms/pin-dashboards
Jun 9, 2026
Merged

allow pinning dashboards#260
vkarpov15 merged 3 commits into
mainfrom
IslandRhythms/pin-dashboards

Conversation

@IslandRhythms

Copy link
Copy Markdown
Contributor
image

@IslandRhythms IslandRhythms requested a review from vkarpov15 June 5, 2026 18:25
@vercel

vercel Bot commented Jun 5, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
studio Ready Ready Preview, Comment Jun 9, 2026 9:32pm

@IslandRhythms IslandRhythms linked an issue Jun 5, 2026 that may be closed by this pull request

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds “pin/unpin” support for dashboards so pinned dashboards appear first in the dashboards list, with a UI control to toggle pin state and backend persistence.

Changes:

  • Add isPinned field to the dashboard model and allow updating it via Dashboard.updateDashboard.
  • Add pin/unpin button in the dashboards list UI and client-side sorting to show pinned dashboards first (including when filtering).
  • Update dashboards listing sort behavior to incorporate pinning.

Reviewed changes

Copilot reviewed 5 out of 7 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
frontend/src/dashboards/dashboards.js Sort dashboards with pinned items first and add togglePin() to persist pin state.
frontend/src/dashboards/dashboards.html Add pin/unpin icon button to each dashboard row.
frontend/public/images/pin.svg Add “pin” icon asset.
frontend/public/images/pin-off.svg Add “unpin” icon asset.
backend/db/dashboardSchema.js Add isPinned field; adjust legacy createdAt backfill logic in post-query middleware.
backend/actions/Dashboard/updateDashboard.js Make code optional; allow updating isPinned.
backend/actions/Dashboard/getDashboards.js Update sort order to include pinned dashboards first.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread frontend/src/dashboards/dashboards.html
Comment thread backend/db/dashboardSchema.js
Comment thread backend/db/dashboardSchema.js
Comment thread backend/actions/Dashboard/getDashboards.js
Comment thread backend/actions/Dashboard/updateDashboard.js
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@vkarpov15 vkarpov15 merged commit 78b08e3 into main Jun 9, 2026
29 of 30 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.

Pin dashboards

3 participants