Skip to content

v1.5.1#43

Merged
b-at-neu merged 4 commits intomainfrom
dev
Feb 24, 2026
Merged

v1.5.1#43
b-at-neu merged 4 commits intomainfrom
dev

Conversation

@b-at-neu
Copy link
Copy Markdown
Collaborator

This pull request introduces a new readOnly mode to both the AdminDashboard and NominationsManager components, ensuring that when a cycle is inactive, users cannot perform actions that modify data. This helps prevent unintended changes during inactive cycles and improves the integrity of the nomination process. The readOnly prop is propagated from the cycle dashboard based on the cycle's active status, and UI elements for destructive or modifying actions are hidden or disabled accordingly.

Read-only mode for inactive cycles:

  • Added a readOnly prop to AdminDashboard and NominationsManager, defaulting to false, and passed it from CycleDashboard based on cycle.isActive. When readOnly is true, UI elements for destructive or modifying actions (such as removing PDFs, rejecting nominations, or bulk approving) are hidden or disabled. [1] [2] [3] [4] app/admin/cycles/[id]/cycle-dashboard.tsxR106-R114)

  • Updated the rendering logic in both components to conditionally display or hide action buttons, checkboxes, and instructional text based on the readOnly state, preventing users from making changes in inactive cycles. [1] [2] [3] [4] [5] [6] [7]

Version bump:

  • Incremented the package version from 1.5.0 to 1.5.1 in package.json to reflect the new read-only functionality.

Copilot AI and others added 4 commits February 24, 2026 05:37
Co-authored-by: b-at-neu <144247685+b-at-neu@users.noreply.github.com>
@b-at-neu b-at-neu self-assigned this Feb 24, 2026
@vercel
Copy link
Copy Markdown

vercel bot commented Feb 24, 2026

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

Project Deployment Actions Updated (UTC)
senate-path Canceled Canceled Feb 24, 2026 6:27am

@b-at-neu b-at-neu merged commit 7f4b3b9 into main Feb 24, 2026
2 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.

2 participants