URL paths - taxonomy and product pages with ACFs and woocommerce products

I have a woocommerce website that has acf pro, bricks builder.

What I am trying to achieve is:

  1. Brands
  2. Models

Post types as they have metadata and I’ll like to make sure they have a relationship.

So Brands > Models

Each Model can have a list of products which are basically color bottles for that model. Each modal also has a form group since they display data about the modal.

What I did was, please tell me if I did it incorrectly but what I did was :

While I have Brands and Modals as post types.

I also created taxonomies for brands which hiearachy parent and modals as the children of this. This is done so the wp grid builder which is for searching would work. I linked this to the products.

However I wasn’t sure on the urls.

I’ll like to have

/brands/ → which is an archive of all the brands logo that each redirect to their brand page.

/brands/brand —> which is a brand page with the field form group that has the information of the brand and a list of modals in it.

/brands/brand/modal ----> which is the modal page which the user selected from the previous page. It has information about the modal and a product list of it.

/brands/brand/modal/colorbottle-1 —> which is the product of this modal.

Does my approach make sense?
Shall I also link the taxonomies of brands to the brand post type and modal post type?

Shall I modify the permalink settings?