search

ynput / ayon-documentation

Documentation of Ayon
Creative Commons Attribution Share Alike 4.0 International
5 stars 13 forks source link

Houdini:add product types table #163

Closed MustafaJafar closed 4 months ago

MustafaJafar commented 4 months ago

Changelog Description

Adding a list of stock product types that come with Ynput's Houdini addon.

Here's a preview: image

MustafaJafar commented 4 months ago

Hey @BigRoy , Do you have any opinion that you would like to share with us regarding this PR?

MustafaJafar commented 4 months ago

Only thing that stood out to me is usage of PointCache instead of Pointcache - not sure if we need the uppercase C there.

I copied pasted the label and product_type

https://github.com/ynput/ayon-core/blob/df976a4ab8900b14c70331519afd563e871c8d09/client/ayon_core/hosts/houdini/plugins/create/create_bgeo.py#L12-L13

whether we're able to keep it sync with the addon nicely and in a timely manner is hard to say.

That's a good question!

MustafaJafar commented 4 months ago

mentioning @antirotor .. This PR is meant to align with https://github.com/ynput/ayon-core/pull/9 to help admins know which values to use with which product types.

mkolar commented 4 months ago

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

MustafaJafar commented 4 months ago

This is no longer needed.