Skip to content

refactor: use data-loader-api internal endpoints, remove KBC_TOKEN#33

Merged
ErikZigo merged 4 commits into
1.4.xfrom
erik-AJDA-2293
Feb 25, 2026
Merged

refactor: use data-loader-api internal endpoints, remove KBC_TOKEN#33
ErikZigo merged 4 commits into
1.4.xfrom
erik-AJDA-2293

Conversation

@ErikZigo

@ErikZigo ErikZigo commented Feb 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Switch saveFile() and updateApiTimestamp() to use /data-loader-api/internal/save and /data-loader-api/internal/activity endpoints
  • Remove KBC_TOKEN / X-StorageApi-Token dependency from autosave flow
  • Remove getStorageTokenFromEnv() function
  • Backport of 2.x data-loader changes to 1.3.2 base (version 1.4.0.dev1)

Test plan

  • All 6 existing tests pass with updated mocks
  • Lint clean (flake8 critical errors)
  • Deploy to test sandbox environment and verify autosave works
    • 3.1.0-5ef6f07-utils-1.4.0.dev1

🤖 Generated with Claude Code

…pendency

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@linear

linear Bot commented Feb 24, 2026

Copy link
Copy Markdown

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Comment thread setup.py Outdated
setup(
name='keboola-sandboxes-notebook-utils',
version='1.3.2',
version='1.4.0.dev1',

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

before merge change to 1.4.0

@ErikZigo ErikZigo requested a review from odinuv February 25, 2026 07:15
@ErikZigo

ErikZigo commented Feb 25, 2026

Copy link
Copy Markdown
Contributor Author

Comment thread setup.py Outdated
@ErikZigo ErikZigo removed the request for review from odinuv February 25, 2026 07:40
Comment thread setup.py Outdated
@ErikZigo ErikZigo merged commit 766a2d2 into 1.4.x Feb 25, 2026
1 check passed
@ErikZigo ErikZigo deleted the erik-AJDA-2293 branch February 25, 2026 18:36
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