Skip to content

fix: mobileVisibleFields signature#1058

Merged
ThibaudDauce merged 1 commit intomainfrom
fix/mobileVisibleFields
Apr 27, 2026
Merged

fix: mobileVisibleFields signature#1058
ThibaudDauce merged 1 commit intomainfrom
fix/mobileVisibleFields

Conversation

@abulte
Copy link
Copy Markdown
Contributor

@abulte abulte commented Apr 27, 2026

The error leaks in downstream projects with stricter typechecking settings:

❯ pnpm run type-check

> udata-front-kit@0.4.0 type-check /Users/alexandre/Developer/Ecolab/Ecopshères/ecospheres-front
> vue-tsc --noEmit --project tsconfig.app.json

node_modules/@datagouv/components-next/src/components/TabularExplorer/TabularExplorer.vue:754:30 - error TS6133: 'row' is declared but its value is never read.

754 function mobileVisibleFields(row: TabularRow, index: number): string[] {
                                 ~~~


Found 1 error in node_modules/@datagouv/components-next/src/components/TabularExplorer/TabularExplorer.vue:754

 ELIFECYCLE  Command failed with exit code 1.

@abulte abulte requested a review from ThibaudDauce April 27, 2026 09:36
@ThibaudDauce ThibaudDauce merged commit eb2fe2c into main Apr 27, 2026
15 checks passed
@ThibaudDauce ThibaudDauce deleted the fix/mobileVisibleFields branch April 27, 2026 10:20
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.

2 participants