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

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

Open
2 tasks done
MustafaJafar opened this issue Feb 28, 2024 · 1 comment
Open
2 tasks done

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

MustafaJafar opened this issue Feb 28, 2024 · 1 comment
Assignees
Labels
type: enhancement Improvement of existing functionality or minor addition

Comments

@MustafaJafar
Copy link
Contributor

MustafaJafar commented Feb 28, 2024

Is there an existing issue for this?

  • I have searched the existing issues.

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

While testing ynput/ayon-core#113 a conflict happened with Ayon parameters as one of the parameters productName was used natively in Karma ROP node. more info read ynput/ayon-core#113 (comment)

So, the problem was solved but we should consider prefixing all parameters as mentioned by @BigRoy
ynput/ayon-core#115 (comment)

How would you imagine the implementation of the feature?

Here's an example by BigRoy
ynput/ayon-core#123 (comment)

Are there any labels you wish to add?

  • I have added the relevant labels to the enhancement request.

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.
ynput/ayon-core#123
ynput/ayon-core#124

[cuID:AY-2542]

@MustafaJafar MustafaJafar added the type: enhancement Improvement of existing functionality or minor addition label Feb 28, 2024
@BigRoy BigRoy transferred this issue from ynput/ayon-core Jul 3, 2024
@antirotor
Copy link
Member

Need some update on this as both PRs mentioned in description were closed without merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants