-
Notifications
You must be signed in to change notification settings - Fork 12
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 product types table #163
Conversation
Hey @BigRoy , |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suppose having a list is helpful - whether we're able to keep it sync with the addon nicely and in a timely manner is hard to say.
Only thing that stood out to me is usage of PointCache
instead of Pointcache
- not sure if we need the uppercase C
there.
Probably best left to someone else to approve it that can look at this from a higher POV. :)
I copied pasted the
That's a good question! |
mentioning @antirotor .. This PR is meant to align with ynput/ayon-core#9 to help admins know which values to use with which product types. |
I made a comment on the corresponding PR as well. I'm not sure this is that useful. We have a page with all the product types in the global docs section and then attributes on the addon in the docs that is visible here https://ayon.ynput.io/features?addons=houdini update that, but don't add the same information into a new table |
This is no longer needed. |
Changelog Description
Adding a list of stock product types that come with Ynput's Houdini addon.
Here's a preview: