Skip to content

Feat: refactor with sorting fix - #28

Merged
Akalanka47000 merged 9 commits into
developmentfrom
feature/refactor-with-sorting-fix
Nov 29, 2025
Merged

Feat: refactor with sorting fix#28
Akalanka47000 merged 9 commits into
developmentfrom
feature/refactor-with-sorting-fix

Conversation

@icy-r

@icy-r icy-r commented Nov 29, 2025

Copy link
Copy Markdown
Member

This pull request introduces a bulk enable/disable feature for questions, along with several UI and filter improvements. The main changes include adding a dialog for bulk status updates, updating the question list UI to reflect question status, and enhancing filtering and sorting options for both questions and submissions.

Bulk Question Status Management:

  • Added a new BulkStatusDialog component for enabling or disabling all questions at once, with integration into the questions page and Redux state management. [1] [2] [3] [4] [5] [6] [7]
  • Implemented a new API mutation and Redux logic for bulk updating question statuses. [1] [2]

UI Improvements:

  • Updated the question card to display whether a question is enabled or disabled.
  • Added "Disable All" and "Enable All" buttons for admins on the questions page.

Filtering and Sorting Enhancements:

  • Added a "Status" filter to question filters, allowing filtering by enabled/disabled status.
  • Added a "Grading Type" filter and a "last updated" sort option to submission filters. [1] [2]
  • Improved default sort directions and added a "Sort by name" option for questions.

Minor Fixes and Improvements:

  • Improved role validation and error messaging in the user dialog.
  • Corrected toast messaging for grading submissions.
  • Fixed a dependency array issue in the authentication hook.

@vercel

vercel Bot commented Nov 29, 2025

Copy link
Copy Markdown

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

Project Deployment Preview Comments Updated (UTC)
bashaway-admin-portal Ready Ready Preview Comment Nov 29, 2025 8:32am

@icy-r
icy-r requested a review from Akalanka47000 November 29, 2025 08:32
@icy-r icy-r self-assigned this Nov 29, 2025
@Akalanka47000
Akalanka47000 changed the base branch from main to development November 29, 2025 12:57
@Akalanka47000
Akalanka47000 merged commit 458808c into development Nov 29, 2025
5 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.

4 participants