Skip to content

Conversation

misterbrunet
Copy link
Contributor

Summary

Adding two new functions for the MagritteCoordinatorClient:

  • api_add_extract_for_branch - returns a basic request.Response from POST "build/add-extract?branchName={branch_name}".
  • create_new_extract_for_source - a wrapper for the function above which returns a str value of the newly created extract/source RID.

Attaching three screenshots from my Foundry ecosystem proving that I was able to obtain a single new extract/source RID using the create_new_extract_for_source function & how such newly created sync looks like.

  1. Getting the export/sync RID in-code (during a debug session):
source_rid_retrieved
  1. Overview of the new export present within the source object on Foundry:
export_1
  1. New export RID matches the one obtained in code:
export_2

Checklist

  • You agree with our CLA
  • Included tests (or is not applicable).
  • Updated documentation (or is not applicable).
  • Used pre-commit hooks to format and lint the code.

@nicornk nicornk requested a review from Copilot October 16, 2025 12:11
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Adds functionality to create a new extract (sync) for a source via MagritteCoordinatorClient.

  • Adds api_add_extract_for_branch to POST build/add-extract.
  • Adds create_new_extract_for_source wrapper constructing payload and returning JSON response.
  • Extends imports with DatasetRid and TransactionType.

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

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.

1 participant