Skip to content

Bundles: Allow to pass project bundle name to bundle methods#307

Open
iLLiCiTiT wants to merge 1 commit intodevelopfrom
enhancement/support-project-bundle-name-in-bundles
Open

Bundles: Allow to pass project bundle name to bundle methods#307
iLLiCiTiT wants to merge 1 commit intodevelopfrom
enhancement/support-project-bundle-name-in-bundles

Conversation

@iLLiCiTiT
Copy link
Member

Changelog Description

Bundle methods can expect project bundle name to fill project bundle name query value.

Additional review information

With project bundles this should be the way how to use bundle endpoints. Currently AYON launcher has own implementation of these methods to solve the issue.

Testing notes:

  1. It is possible to pass project_bundle_name to bundle related methods to get the project bundle specific data.

@iLLiCiTiT iLLiCiTiT added the type: enhancement New feature or request label Feb 25, 2026
@iLLiCiTiT iLLiCiTiT self-assigned this Feb 25, 2026
Copy link
Contributor

@BigRoy BigRoy left a comment

Choose a reason for hiding this comment

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

Load works fine.

But create_link logic does not seem work at all.

Comment on lines 6904 to 6943
@@ -6906,7 +6921,8 @@ def create_link(
output_id (str): Output entity id.
output_type (str): Entity type of output entity.
link_name (Optional[str]): Name of link.
Available from server version '1.0.0-rc.6'.
data (Optional[dict[str, Any]]): Additional data to be stored
with the link.

Returns:
CreateLinkData: Information about link.
@@ -6924,6 +6940,7 @@ def create_link(
output_id=output_id,
output_type=output_type,
link_name=link_name,
data=data,
Copy link
Contributor

Choose a reason for hiding this comment

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

This PR seems to include: #308

Copy link
Contributor

Choose a reason for hiding this comment

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

Overall PR seems to work, except for this change?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants