Your model looks fine but it brings us back to the classic tale of a star- and snowflake schema…
Here’s some more information about that:
As far as your relationships go they are all single direction and one to many - no problems there.
Visually it would make more sense to place the product category and product subcategory tables above the product table.
That’s it for me.