Skip to content

Add support for representation traits #225

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 19 commits into from
Apr 14, 2025

Conversation

antirotor
Copy link
Member

@antirotor antirotor commented Dec 16, 2024

Changelog Description

AYON server added DB support for traits as of 1.6.0, this is adding simple way to work with them. GraphQL support on the server side was added in 1.7.5

For more information about traits see ynput/ayon-core#979

Additional review information

Warning

This needs ynput/ayon-backend#466

@antirotor antirotor added the type: enhancement New feature or request label Dec 16, 2024
@antirotor antirotor self-assigned this Dec 16, 2024
@antirotor antirotor requested a review from iLLiCiTiT December 16, 2024 16:07
@iLLiCiTiT iLLiCiTiT requested review from kalisp and removed request for kalisp December 17, 2024 09:10
antirotor and others added 3 commits March 28, 2025 18:39
add traits as the default field for the representation
…entation-fields' into feature/add-support-for-representation-traits
this is to maintain compatibility with older server versions
@antirotor antirotor marked this pull request as ready for review April 9, 2025 09:13
@iLLiCiTiT
Copy link
Member

iLLiCiTiT commented Apr 11, 2025

Changed order of traits argument to be after tags (at most places). And avoid adding traits in new representation if are empty.

EDIT: Maybe representation_traits_available should be renamed back to grahql_allows_traits_in_representations. Or the version in the method does not match the version when the traits were added to server.

@antirotor antirotor merged commit 8731265 into develop Apr 14, 2025
2 checks passed
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