Skip to content

Add workflow_task to Run, improve docs - #94

Merged
labkey-alan merged 23 commits into
developfrom
fb_assay_run_workflow_task
Aug 6, 2026
Merged

labkey-alan merged 23 commits into
developfrom
fb_assay_run_workflow_task

Conversation

@labkey-alan

@labkey-alan labkey-alan commented Aug 4, 2026

Copy link
Copy Markdown
Contributor

Rationale

This PR addresses two issues LabKey/internal-issues#1068, and LabKey/internal-issues#1195, adds improved documentation for our ExperimentWrapper and QueryWrapper, and adds the import_run API to the ExperimentWrapper.

Related Pull Requests

  • n/a

Changes

  • Add workflow_task to Run
    • Resolves LabKey/internal-issues#1068
  • Add import_run to ExperimentWrapper
  • Add timeout arg to import_rows
    • Resolves LabKey/internal-issues#1195
  • Add docs for QueryWrapper
    • docs/query.md
  • Delete samples/query_examples.py
  • Add docs for ExperimentWrapper
    • docs/experiment.md
  • Delete samples/experiment_examples.py
  • query.import_rows: add the params that were missing from the server's import action
    • audit_details, audit_user_comment, format, import_identity, import_url, module, module_resource, path, save_to_pipeline, text, use_async
    • data_file is now optional, the rows may instead come from text, path, or module_resource
    • Add InsertOption class enumerating the values accepted by insert_option

@labkey-alan labkey-alan self-assigned this Aug 4, 2026

@labkey-nicka labkey-nicka 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.

Nice. The clean_payload utility is certainly an improvement for readability.

@labkey-alan
labkey-alan merged commit c8e0a51 into develop Aug 6, 2026
2 of 3 checks passed
@labkey-alan
labkey-alan deleted the fb_assay_run_workflow_task branch August 6, 2026 18:04
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