Skip to content

Remove react-data-table from file connection preview#3054

Merged
arsen-afaunov merged 1 commit intomainfrom
CHARTS-10721-remove-react-data-table-from-connection-preview
Oct 28, 2025
Merged

Remove react-data-table from file connection preview#3054
arsen-afaunov merged 1 commit intomainfrom
CHARTS-10721-remove-react-data-table-from-connection-preview

Conversation

@arsen-afaunov
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented Oct 22, 2025

📦 Statoscope quick diff with main-branch:

⏱ Build time: 583 ms (1.08%)

⚖️ Initial size: 0.15 kb (0%)

🕵️ Validation errors: 0

Full Statoscope report could be found here ↗️

@github-actions
Copy link

github-actions bot commented Oct 22, 2025

E2E Tests

Playwright test results

passed  181 passed
flaky  7 flaky
skipped  2 skipped

Details

stats  190 tests across 105 suites
duration  3 minutes, 16 seconds
commit  2dffd18

Flaky tests

opensource › opensource-suites/connections/postgres.test.ts › Postgres connector interactions › Connector should be created
opensource › opensource-suites/dash/revisions/revisionsListWithCreationDashboards.test.ts › Dashboards - Versioning › Creating a dashboard, editing, saving as a draft, making the draft version relevant
opensource › opensource-suites/wizard/filters/filters-with-dialog-color.test.ts › Wizard filters › Two or more values of the Date field from the dashboard filter section should use IN operation by default
opensource › opensource-suites/wizard/visualizations/flat-table/linear-indicator.test.ts › Wizard › Flat table › Linear indicator with manual scale(the set max value is less than the actual) @screenshot
opensource › opensource-suites/wizard/visualizations/metric/title.test.ts › Wizard › Metric chart › Title setting
opensource › opensource-suites/wizard/visualizations/pivot-table/export.test.ts › Wizard › Pivot table › Export to xlsx
opensource › opensource-suites/wizard/visualizations/pivot-table/sorting.test.ts › Wizard › Pivot table › Click on last row header cell should sort columns

Skipped tests

opensource › opensource-suites/dash/revisions/revisionsList.test.ts › Dashboards - Versioning › Dashboard with a long list of revisions, checking the upload and the updated list of revisions
opensource › opensource-suites/dash/selectors/autoupdateRelatedSelectors.test.ts › Dashboards - Autoupdate options of group selectors › Manual selectors with aliases don't affect each other before applying when auto-update is disabled


🧐 Links with traces:

👉 Link with detailed report: report

🐳 Link with docker compose logs: logs

Last updated: 2025-10-28 11:44:39 MSK

@github-actions
Copy link

github-actions bot commented Oct 22, 2025

E2E Tests Farm

Playwright test results

passed  419 passed
flaky  6 flaky
skipped  1 skipped

Details

stats  426 tests across 171 suites
duration  5 minutes, 18 seconds
commit  2dffd18

Flaky tests

basic › suites/charts/export/forbiddenExport.test.ts › Chart export availability › Preview with forbidden export › Ql preview - Export menu item is disabled if export is forbidden on connection
basic › suites/charts/export/forbiddenExport.test.ts › Chart export availability › Ql - Export menu item is disabled if export is forbidden on connection
basic › suites/dataset/parameters/parameterDialogValidation.test.ts › Datasets - Parameter dialog validation › Required fields validation
basic › suites/dataset/parameters/parameterDialogValidation.test.ts › Datasets - Parameter dialog validation › Parameter name validation
basic › suites/dataset/parameters/parameterDialogValidation.test.ts › Datasets - Parameter dialog validation › Parameter default_value validation [number]
basic › suites/wizard/placeholder-settings/autoScale.test.ts › Wizard - placeholder dialog ("Autoscaling") › If there is a field in "Y" and "Y2", then you can adjust the scaling for each axis separately

Skipped tests

basic › suites/wizard/combined-chart/layers-and-multidatasets.test.ts › Multi-faceted charts with layers › The data on the X-axis must be formed correctly from two datasets


👉 Link with detailed report: report

Last updated: 2025-10-28 11:49:33 MSK

@arsen-afaunov arsen-afaunov force-pushed the CHARTS-10721-remove-react-data-table-from-connection-preview branch from afe682e to 71e0931 Compare October 23, 2025 16:21
@github-actions
Copy link

github-actions bot commented Oct 23, 2025

E2E Tests No Auth

Playwright test results

passed  185 passed
flaky  3 flaky
skipped  2 skipped

Details

stats  190 tests across 105 suites
duration  2 minutes, 59 seconds
commit  2dffd18

Flaky tests

opensource › opensource-suites/dash/revisions/revisionsListWithCreationDashboards.test.ts › Dashboards - Versioning › Creating a dashboard, editing, saving as a draft, making the draft version relevant
opensource › opensource-suites/wizard/filters/filters-with-dialog-color.test.ts › Wizard filters › Two or more values of the Date field from the dashboard filter section should use IN operation by default
opensource › opensource-suites/wizard/visualizations/bar-y/sorting.test.ts › Wizard › Bar-y chart › Sorting by X field @screenshot

Skipped tests

opensource › opensource-suites/dash/revisions/revisionsList.test.ts › Dashboards - Versioning › Dashboard with a long list of revisions, checking the upload and the updated list of revisions
opensource › opensource-suites/dash/selectors/autoupdateRelatedSelectors.test.ts › Dashboards - Autoupdate options of group selectors › Manual selectors with aliases don't affect each other before applying when auto-update is disabled


🧐 Links with traces:

👉 Link with detailed report: report

🐳 Link with docker compose logs: logs

Last updated: 2025-10-28 11:43:46 MSK

@arsen-afaunov arsen-afaunov force-pushed the CHARTS-10721-remove-react-data-table-from-connection-preview branch 4 times, most recently from 670b13f to 67c6ecc Compare October 27, 2025 12:27
kuzmadom
kuzmadom previously approved these changes Oct 27, 2025
@arsen-afaunov arsen-afaunov force-pushed the CHARTS-10721-remove-react-data-table-from-connection-preview branch 2 times, most recently from 3bfe183 to 0438ccb Compare October 28, 2025 08:07
@arsen-afaunov arsen-afaunov force-pushed the CHARTS-10721-remove-react-data-table-from-connection-preview branch from 0438ccb to 2dffd18 Compare October 28, 2025 08:39
@arsen-afaunov arsen-afaunov merged commit 3b29b77 into main Oct 28, 2025
13 checks passed
@arsen-afaunov arsen-afaunov deleted the CHARTS-10721-remove-react-data-table-from-connection-preview branch October 28, 2025 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants