Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

AYON: Prepare for 'data' via graphql #5923

Merged
merged 2 commits into from
Nov 20, 2023
Merged

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

AYON server does support to query 'data' field for hierarchy entities (project > ... > representation) using GraphQl since version 0.5.5. Because of this PR in ayon-python-api it is required to modify custom graphql function in openpype.client to support that option.

Additional info

This is a preparation/future compatibility for new ayon-python-api version. The issue with "data" queried via graphlql is that it returns None or json string and the json string must be converted.

Testing notes:

At this moment everything should work as before.

@ynbot ynbot added type: feature Larger, user affecting changes and completely new things size/XS Denotes a PR changes 0-99 lines, ignoring general files labels Nov 15, 2023
@iLLiCiTiT iLLiCiTiT changed the title AYON: Prepare AYON: Prepare for 'data' via graphql Nov 15, 2023
Copy link
Member

@kalisp kalisp left a comment

Choose a reason for hiding this comment

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

Doesn't seem to explode anything, so probably ok.

@iLLiCiTiT iLLiCiTiT merged commit 8348fe9 into develop Nov 20, 2023
17 checks passed
@iLLiCiTiT iLLiCiTiT deleted the feature/data-in-graphql branch November 20, 2023 10:02
@ynbot ynbot added this to the next-patch milestone Nov 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/XS Denotes a PR changes 0-99 lines, ignoring general files type: feature Larger, user affecting changes and completely new things
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants