Skip to content

[JSX] Convert FilterableDataTable to Functional Component#10312

Open
HenriRabalais wants to merge 6 commits intoaces:mainfrom
HenriRabalais:2026-01-27_convert-filterabledatatable-to-functional-component
Open

[JSX] Convert FilterableDataTable to Functional Component#10312
HenriRabalais wants to merge 6 commits intoaces:mainfrom
HenriRabalais:2026-01-27_convert-filterabledatatable-to-functional-component

Conversation

@HenriRabalais
Copy link
Collaborator

@HenriRabalais HenriRabalais commented Jan 27, 2026

Description

This PR performs a migration for the FilterableDataTable class component to a functional component.

Key Changes

1. TypeScript Migration & Architecture.

2. Logic & State Improvements

How to Test

  1. Initial Load: Navigate to the table; it should load with no filters.
  2. Filtering: Apply a filter (e.g., Search Name). Verify:
    • The table filters immediately.
    • The URL updates with ?filters=... after the 300ms debounce.
  3. Persistence: Refresh the browser with active filters in the URL. Verify the filters are automatically reapplied.
  4. Clearing: Click the "Clear All" button. Verify the table resets and the URL parameters are removed.

@github-actions github-actions bot added the Language: Javascript PR or issue that update Javascript code label Jan 27, 2026
@HenriRabalais HenriRabalais changed the title [JSX] Migrate FilterableDataTable to TypeScript and Centralize Table Types [JSX] Convert FilterableDataTable to Functional Component Jan 27, 2026
include removed and altered files

removed imports

passing lint

converting progressbar and filter to tsx and moving type declarations

trying to convert Filter to tsx
@HenriRabalais HenriRabalais force-pushed the 2026-01-27_convert-filterabledatatable-to-functional-component branch from 0320096 to 217fc12 Compare March 12, 2026 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Language: Javascript PR or issue that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants