ynput / ayon-core

Apache License 2.0
25 stars 30 forks source link

Remove deprecated `ocio_config` `ImageIOConfigModel` settings for each addon #785

Closed BigRoy closed 1 month ago

BigRoy commented 1 month ago

Is there an existing issue for this?

Please describe the feature you have in mind and explain what the current shortcomings are?

Currently the settings for ayon-houdini contains a deprecated ImageIOConfigModel.

With the support of profiles to determine OCIO config in ayon-core this logic has been superseded and should be removed from each addon still having the deprecated settings.

How would you imagine the implementation of the feature?

Remove the deprecated settings, and ensure transition for existing users is easy. Since settings have moved, in a potentially backwards incompatible way, there may need to be a transition guide with the release explaining what to do. We may need to also bump the minor version to indicate the breaking changes.

Are there any labels you wish to add?

Describe alternatives you've considered:

No response

Additional context:

Part of the cleanup epic ✨

BigRoy commented 1 month ago

Oops - can we link multiple PRs but have this only close if ALL of them are merged? If not, I may need to unlink them and just keep them in the PR description. @dee-ynput do you know?

dee-ynput commented 1 month ago

AFAIK closing one will always close the issue :'(

Mentioning them in the issue body is better in this case (and probably most cases)

I hope GH will change that someday ✨

BigRoy commented 1 month ago

@dee-ynput all the listed integrations are now merged - I'll close this one.