Skip to content

python(feat): list and get data imports - #767

Merged
wei-sift merged 2 commits into
mainfrom
eng-14183-add-list-to-the-sift-client-dataimportapi
Aug 31, 2026
Merged

wei-sift merged 2 commits into
mainfrom
eng-14183-add-list-to-the-sift-client-dataimportapi

Conversation

@wei-sift

@wei-sift wei-sift commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds read access to data imports in sift_client

  • client.data_import.list_(), get(), and find(), including a DataImport type with the import's status, error, and warnings.
  • run.data_imports: a property that lists imports for a run directly.
  • job.get_data_import(): get the import from the job import_from_path returns.

Testing

  • Unit tests
  • Manually listed imports by run, fetched by ID, and verified run.data_imports returns the same set.

@github-actions

github-actions Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

Python docs preview: https://sift-stack.github.io/sift/python/pr-767/

Deployed from da426a2. The link may take up to a minute to become live as GitHub Pages propagates.

@wei-sift
wei-sift force-pushed the eng-14183-add-list-to-the-sift-client-dataimportapi branch from b0c05aa to 47750fe Compare August 31, 2026 05:12
@wei-sift
wei-sift requested a review from nathan-sift August 31, 2026 05:16
@wei-sift
wei-sift marked this pull request as ready for review August 31, 2026 05:16
Comment thread python/lib/sift_client/sift_types/data_import.py

@nathan-sift nathan-sift left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good!

@wei-sift
wei-sift merged commit 26cba3d into main Aug 31, 2026
31 of 32 checks passed
@wei-sift
wei-sift deleted the eng-14183-add-list-to-the-sift-client-dataimportapi branch August 31, 2026 20:22
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