Skip to content
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

Add entityTraits support #93

Merged
merged 2 commits into from
Dec 8, 2023
Merged

Conversation

feltech
Copy link
Member

@feltech feltech commented Nov 30, 2023

Closes #89. The entityTraits API method is a new required method.

@feltech feltech requested a review from a team as a code owner November 30, 2023 09:11
@feltech feltech marked this pull request as draft November 30, 2023 09:18
@feltech feltech force-pushed the work/89-entityTraits branch from d19146b to afec031 Compare November 30, 2023 11:53
@feltech feltech marked this pull request as ready for review November 30, 2023 11:56
@feltech feltech self-assigned this Nov 30, 2023
Copy link
Contributor

@foundrytom foundrytom left a comment

Choose a reason for hiding this comment

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

Nice to see how easy it was to add...

@feltech feltech force-pushed the work/89-entityTraits branch 3 times, most recently from 921b7af to db143b6 Compare December 4, 2023 20:01
@foundrytom foundrytom self-requested a review December 8, 2023 11:52
@feltech feltech force-pushed the work/89-entityTraits branch from db143b6 to 35d19e4 Compare December 8, 2023 12:25
Copy link
Contributor

@foundrytom foundrytom left a comment

Choose a reason for hiding this comment

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

Epic, thanks!!!

@foundrytom foundrytom self-requested a review December 8, 2023 12:37
Closes OpenAssetIO#89. The `entityTraits` API method is a new required method.

For reading, return the traits of the entity, plus the implicit
VersionTrait. For write, return the traits of the entity, but don't
include the VersionTrait (since it cannot be written directly).

The implementation assumes new entities have no restriction on the
traits that can be published to them; whereas new versions of existing
entities are restricted to the same trait set as previous versions.
This latter case is not actually enforced during `preflight`/`register`.
See OpenAssetIO#92.

Signed-off-by: David Feltell <[email protected]>
Part of OpenAssetIO#89. Until the next OpenAssetIO release, we must once again use
wheels built from the main branch, and cannot have a package dependency
on the unreleased version.

Signed-off-by: David Feltell <[email protected]>
@feltech feltech force-pushed the work/89-entityTraits branch from 27fe84e to cdf18c1 Compare December 8, 2023 12:40
@feltech feltech merged commit d48400f into OpenAssetIO:main Dec 8, 2023
5 checks passed
@feltech feltech deleted the work/89-entityTraits branch December 8, 2023 15:02
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.

Add entityTraits
3 participants