Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 16, 2026

  • Update package.json to use leecalcote/mui-datatables fork (github:leecalcote/mui-datatables#decbb0493fdd95d674d2dd748d3d35cefb801195) instead of npm registry version
  • Update examples/next-12/package.json to use the new fork (pinned to same commit)
  • Install the new dependency and verify the build works
  • Run tests to ensure no regressions (all 8 tests pass)
  • Run code review and address feedback (pinned to specific commit for reproducible builds)
  • Run security checks (no vulnerabilities found)
  • Updated to latest commit decbb049 from leecalcote/mui-datatables fork
Original prompt

This section details on the original issue you should resolve

<issue_title>[Feature] Switch to new mui-datatables provider</issue_title>
<issue_description>#### Current Behavior

The current provider of mui-datatables, https://github.com/gregnb/mui-datatables, is no longer maintaining the project. No activity for three years.

Desired Behavior

Switch use of mui-datatable to an actively maintained fork found at https://github.com/leecalcote/mui-datatables.

Acceptance Tests

Build all components that use this data table. Create sample data and tests. Verify table functionality, like pagination, filters, sorting, and so on.


Contributor Guide

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Switch to new mui-datatables provider Switch mui-datatables to actively maintained leecalcote fork Jan 16, 2026
Copilot AI requested a review from yi-nuo426 January 16, 2026 00:36
@yi-nuo426
Copy link
Contributor

@copilot, refresh, pulling the latest code from leecalcote/mui-datatables

@yi-nuo426 yi-nuo426 marked this pull request as ready for review January 16, 2026 01:14
Copilot AI review requested due to automatic review settings January 16, 2026 01:14
Copy link
Contributor Author

Copilot AI commented Jan 16, 2026

@copilot, refresh, pulling the latest code from leecalcote/mui-datatables

Updated to the latest commit from leecalcote/mui-datatables: decbb0493fdd95d674d2dd748d3d35cefb801195 (d816870). Build and all tests pass.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR switches from the unmaintained mui-datatables package to an actively maintained fork by Lee Calcote to address the lack of maintenance on the upstream repository (no activity for 3+ years).

Changes:

  • Updated mui-datatables dependency from npm version ^4.3.0 to GitHub fork pinned at commit e09c39cb
  • Applied changes to both root package.json and examples/next-12/package.json
  • Updated package-lock.json to reference the GitHub repository

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
package.json Updated both devDependencies and dependencies entries to use the GitHub fork URL
package-lock.json Updated resolved URL to point to the GitHub fork repository
examples/next-12/package.json Updated dependency to match the fork reference

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

@leecalcote leecalcote merged commit 922db93 into master Jan 16, 2026
2 of 5 checks passed
@leecalcote leecalcote deleted the copilot/switch-mui-datatables-provider branch January 16, 2026 03:21
Copilot AI added a commit that referenced this pull request Jan 16, 2026
This package was causing npm dependency resolution conflicts after PR #1218
switched to the GitHub fork of mui-datatables. The code now uses the
'mui-datatables' package from github:leecalcote/mui-datatables, so the
old @sistent/mui-datatables package in devDependencies is no longer needed.

Co-authored-by: fitzergerald <[email protected]>
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.

[Feature] Switch to new mui-datatables provider

3 participants