Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update table #606

Merged
merged 9 commits into from
May 20, 2023
Merged

Update table #606

merged 9 commits into from
May 20, 2023

Conversation

JanMarvin
Copy link
Owner

@JanMarvin JanMarvin commented May 10, 2023

Parts of this should cover #105

  • wb_remove_tables() should not throw errors and should only remove the table attributes on a sheet and in the workbook
  • this way it can be added to delete_data() or wb_clean_sheet() (and maybe the latter should replace the former)
  • wb_update_table() is a manual way to modify the table xml structure (not sure if this covers every possible xml element and obviously it does not alter the autofilter)
  • TODO: wb_get_named_regions() as wrapper

R/get-named-regions.R Outdated Show resolved Hide resolved
@JanMarvin JanMarvin merged commit 9d3d887 into main May 20, 2023
9 checks passed
@JanMarvin JanMarvin deleted the update_table branch May 20, 2023 14:58
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.

None yet

1 participant