Skip to content

Conversation

yashuatla
Copy link
Owner

@yashuatla yashuatla commented Jun 25, 2025

PR Summary

Refactor DeleteTable Function to Use getTables()

Overview

This PR refactors the DeleteTable function to use the getTables() method instead of iterating through sheets and getting tables for each sheet separately, improving code efficiency.

Change Types

Type Description
Refactor Optimize DeleteTable function to use getTables() for better efficiency

Affected Modules

Module / File Change Description
table.go Refactored DeleteTable function to use getTables() method directly

zhayt and others added 15 commits July 5, 2024 14:34
…nge (qax-os#1941)

- Add unit test for get pivot table with across worksheet data range, update dependencies package and updated comments of the GetMergeCells function
… hyperlink by None linkType

- Update unit tests
…Axis data type (qax-os#1946)

- Introduce new exported ChartTickLabelPositionType enumeration
- Update unit tests
… for zero numeric cell value (qax-os#1947)

- Updated unit tests
…l is empty in pivot table data range

- Update unit tests
…ith an IEEE 754 "not-a-number" value or infinity
- Fix missing conditional formatting after remove column
- Fix the SetSheetVisible function panic on none views sheet
- Updated unit tests
…eld in the PivotTableField data type

- Support set and get built-in number format of the pivot table data filed cells
- Update unit tests
- Fixed ineffectual assignment issue
…ion tags and ID (qax-os#1951)

- Update unit tests for specified date and time format code
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.

10 participants