Skip to content

Update TODO.md to reflect implemented functions#14

Merged
tin900 merged 2 commits into
mainfrom
copilot/fix-13
Jul 18, 2025
Merged

Update TODO.md to reflect implemented functions#14
tin900 merged 2 commits into
mainfrom
copilot/fix-13

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jul 17, 2025

The TODO.md file was out of sync with the actual implemented functions in the repository. Several functions that were already implemented and exported in the NAMESPACE were still marked as incomplete in the TODO checklist.

Changes Made

Updated 4 TODO items from [ ] to [x] to reflect the current state of implemented functions:

  • Line 70: Update Data Source Now (update_data_source_now)
  • Line 83: Run Extract Refresh Task (run_extract_refresh_task)
  • Line 268: Server Info (get_server_info)
  • Line 339: Update Workbook Now (update_workbook_now)

Verification

  • Total TODO items: 340 (unchanged)
  • Completed items: 36 (increased from 32)
  • Remaining items: 304 (decreased from 308)

All functions exported in the NAMESPACE that correspond to Tableau REST API methods are now properly marked as completed in the TODO.md file.

Fixes #13.

Warning

Firewall rules blocked me from connecting to one or more addresses

I tried to connect to the following addresses, but was blocked by firewall rules:

  • esm.ubuntu.com
    • Triggering command: /usr/lib/apt/methods/https (dns block)

If you need me to access, download, or install something from one of these locations, you can either:


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Co-authored-by: tin900 <113692500+tin900@users.noreply.github.com>
Copilot AI changed the title [WIP] Update TODO.md Update TODO.md to reflect implemented functions Jul 17, 2025
Copilot AI requested a review from tin900 July 17, 2025 16:41
Copy link
Copy Markdown
Contributor

@tin900 tin900 left a comment

Choose a reason for hiding this comment

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

good.

@tin900 tin900 marked this pull request as ready for review July 18, 2025 05:55
@tin900 tin900 merged commit 1a6995d into main Jul 18, 2025
2 checks passed
@tin900 tin900 deleted the copilot/fix-13 branch July 18, 2025 06:28
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.

Update TODO.md

2 participants