ynput / ayon-houdini

Houdini addon for AYON
Apache License 2.0
6 stars 2 forks source link

Houdini: Add 'AYON_' prefix to all instance keys #7

Open MustafaJafar opened 4 months ago

MustafaJafar commented 4 months ago

Is there an existing issue for this?

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

While testing https://github.com/ynput/ayon-core/pull/113 a conflict happened with Ayon parameters as one of the parameters productName was used natively in Karma ROP node. more info read https://github.com/ynput/ayon-core/pull/113#issuecomment-1961621273

So, the problem was solved but we should consider prefixing all parameters as mentioned by @BigRoy https://github.com/ynput/ayon-core/pull/115#issuecomment-1962657403

How would you imagine the implementation of the feature?

Here's an example by BigRoy https://github.com/ynput/ayon-core/pull/123#discussion_r1504564660

Are there any labels you wish to add?

Describe alternatives you've considered:

No response

Additional context:

These PRs were made to address this issue.. But, I'm making this issue as it took longer than I thought and needed a lot of testing. https://github.com/ynput/ayon-core/pull/123 https://github.com/ynput/ayon-core/pull/124

[cuID:AY-2542]