Skip to content

fix(apollo-vertex): add expanded row in data table#388

Open
alincadariu wants to merge 1 commit intomainfrom
alincadariu/datatable-row-expanded
Open

fix(apollo-vertex): add expanded row in data table#388
alincadariu wants to merge 1 commit intomainfrom
alincadariu/datatable-row-expanded

Conversation

@alincadariu
Copy link
Collaborator

@alincadariu alincadariu commented Mar 23, 2026

i realized we broke the expanded row example with the data table hooks refactors.

Summary

  • Wire up the existing expand column in the DataTable template with expanded state, onExpandedChange, and renderExpandedRow
  • Reorder columns to: select, expand, status, email, amount, actions
  • Fix duplicate React key warning in data-table.tsx by using React.Fragment with key

Test plan

  • Clear localStorage for payments-demo keys
  • Visit the data table page and verify the expand chevron appears after the select checkbox
  • Click the chevron to expand a row and verify payment details are shown
  • Click again to collapse

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link

github-actions bot commented Mar 23, 2026

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

Project Deployment Review Updated (PT)
apollo-canvas 🟢 Ready Preview, Logs Mar 23, 2026, 05:06:56 AM
apollo-landing 🟢 Ready Preview, Logs Mar 23, 2026, 05:03:52 AM
apollo-ui-react 🟢 Ready Preview, Logs Mar 23, 2026, 05:05:33 AM
apollo-vertex 🟢 Ready Preview, Logs Mar 23, 2026, 05:05:01 AM
apollo-wind 🟢 Ready Preview, Logs Mar 23, 2026, 05:03:59 AM

@KokoMilev KokoMilev enabled auto-merge (rebase) March 23, 2026 12:02
@github-actions
Copy link

Dependency License Review

  • 1916 package(s) scanned
  • ✅ No license issues found
  • ⚠️ 3 package(s) excluded (see details below)
License distribution
License Packages
MIT 1681
ISC 89
Apache-2.0 56
BSD-3-Clause 28
BSD-2-Clause 23
BlueOak-1.0.0 8
MPL-2.0 5
MIT OR Apache-2.0 3
MIT-0 3
CC0-1.0 3
LGPL-3.0-or-later 2
(MIT OR Apache-2.0) 2
Unlicense 2
Python-2.0 1
CC-BY-4.0 1
(MPL-2.0 OR Apache-2.0) 1
Unknown 1
Artistic-2.0 1
(WTFPL OR MIT) 1
(BSD-2-Clause OR MIT OR Apache-2.0) 1
CC-BY-3.0 1
0BSD 1
(MIT OR CC0-1.0) 1
MIT AND ISC 1
Excluded packages
Package Version License Reason
@img/sharp-libvips-linux-x64 1.2.4 LGPL-3.0-or-later LGPL pre-built binary, not linked
@img/sharp-libvips-linuxmusl-x64 1.2.4 LGPL-3.0-or-later LGPL pre-built binary, not linked
khroma 2.1.0 Unknown MIT per GitHub repo, missing license field in package.json

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.

1 participant