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
Optimize component naming #67
Comments
That's right! I'll do this simple changes. But.. promoted products are a little bit more complicated. |
If you can do this and #40 together that would be great and prevent some nasty merge conflicts! Leave promoted products for later. |
Alright, sorry for delay but server issues stole the night :) |
I'm was about to change this, but I want to propose first this. Why not like this: Category -> Products Especially changing Product to ProductDetail I see as unnecessary complication. Or you have any real reason for that? I'm not sure about English here, but it seems both ProductFilter and ProductsFilter are correct. But in our case this one is tightly connect to Products component, works as it's extension. So to naming can reflect that. What do you think? |
Good points, let's go with your suggestions 👍 |
New proposed names:
Category
->ProductList
Product
->ProductDetail
CategoryFilter
->ProductFilter
The text was updated successfully, but these errors were encountered: