Skip to content

feat: manage opening status from provider to avoid closing on re-renders#3480

Open
RafaMellado wants to merge 1 commit intomainfrom
nested-table/manage-rows-from-provider
Open

feat: manage opening status from provider to avoid closing on re-renders#3480
RafaMellado wants to merge 1 commit intomainfrom
nested-table/manage-rows-from-provider

Conversation

@RafaMellado
Copy link
Contributor

Using nested tables, we discovered that, when we updated the parent resource (from apollo cache), the row re-rendered and closed automatically the results.

With this change, we want to manage the opening status from outside, and avoid to lose what we interacted with the table on re-renders

@RafaMellado RafaMellado requested a review from a team as a code owner February 19, 2026 09:59
@github-actions github-actions bot added feat react Changes affect packages/react labels Feb 19, 2026

return (
<div className="flex h-full min-h-0 flex-col gap-4">
<OneTable loading={isLoading}>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

All this introducing the Wrapper lol 😂

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

✅ No New Circular Dependencies

No new circular dependencies detected. Current count: 0

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

🔍 Visual review for your branch is published 🔍

Here are the links to:

@github-actions
Copy link
Contributor

github-actions bot commented Feb 19, 2026

Coverage Report for packages/react

Status Category Percentage Covered / Total
🔵 Lines 38.52% 7042 / 18278
🔵 Statements 37.92% 7198 / 18982
🔵 Functions 29.05% 1502 / 5169
🔵 Branches 28.2% 4058 / 14388
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
packages/react/src/experimental/OneDataCollection/visualizations/collection/Table/Table.tsx 81.81% 50.49% 62.5% 82.53% 174-178, 315, 346-401, 424-512
packages/react/src/experimental/OneDataCollection/visualizations/collection/Table/components/NestedRow.tsx 8.51% 0% 0% 8.51% 93-279, 342
packages/react/src/experimental/OneDataCollection/visualizations/collection/Table/providers/NestedProvider.tsx 13.63% 0% 0% 15% 30-74, 79-85
Generated in workflow #11055 for commit 6df0712 by the Vitest Coverage Report Action

@RafaMellado RafaMellado force-pushed the nested-table/manage-rows-from-provider branch from 3ebc7b7 to 6df0712 Compare February 19, 2026 11:19
@github-actions
Copy link
Contributor

📦 Alpha Package Version Published

Use pnpm i github:factorialco/f0#npm/alpha-pr-3480 to install the package

Use pnpm i github:factorialco/f0#83ce5c03434ce1588934abef2cbec6f7b962e99e to install this specific commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat react Changes affect packages/react

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant