Skip to content

chore: remove datatable from groups - #492

Merged
guillaumedc merged 1 commit into
mainfrom
chore/replace-datatables
Jun 8, 2026
Merged

chore: remove datatable from groups#492
guillaumedc merged 1 commit into
mainfrom
chore/replace-datatables

Conversation

@ctresvaux

@ctresvaux ctresvaux commented Jun 5, 2026 •
edited by guillaumedc
Loading

Copy link
Copy Markdown
Contributor

Related issue(s)

#142

How to test manually

  • Populate the group table with at least 10 records.
  • Test pagination.
  • Test search.
  • Test column sorting.

Reviewer checklist

  • Code is manually tested
  • Interface works on both mobiles and big screens
  • Interface works on both Firefox and Chrome
  • Tests are up to date
  • Documentation is up to date
  • Pull request has been reviewed and approved

@ctresvaux
ctresvaux force-pushed the chore/replace-datatables branch from 3a2e751 to 5ac0781 Compare June 5, 2026 08:30
@ctresvaux
ctresvaux force-pushed the chore/replace-datatables branch from 5ac0781 to 8b40f67 Compare June 5, 2026 09:06
@ctresvaux
ctresvaux marked this pull request as draft June 5, 2026 13:05
@ctresvaux
ctresvaux force-pushed the chore/replace-datatables branch 4 times, most recently from 2bcef8b to 2a776d4 Compare June 5, 2026 14:24
@ctresvaux
ctresvaux marked this pull request as ready for review June 5, 2026 14:24
@ctresvaux
ctresvaux force-pushed the chore/replace-datatables branch from 2a776d4 to f847da0 Compare June 8, 2026 09:54
@guillaumedc
guillaumedc merged commit f847da0 into main Jun 8, 2026
3 checks passed
@guillaumedc
guillaumedc deleted the chore/replace-datatables branch June 8, 2026 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@guillaumedc guillaumedc guillaumedc approved these changes
@marien-probesys marien-probesys Awaiting requested review from marien-probesys

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ctresvaux @guillaumedc @ctresvauxp6