-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add Breadcrumbs #26
Comments
@rohrig for this task I've following questions
|
@faizanarif121 Those are very valid points. It looks like this is perhaps not a good first issue after all. 😁 I had a look at V2 and the solution there is not exactly trivial. You can check there for inspiration, but we are by no means bound to anything there. If you find a cleaner, better way to do it, that's great. It looks like there is a separate solution for the category page and the product pages. I'll adjust the issue description accordingly. If you're up for it. You could plan a solution. I'm happy to support you in any way I can. We can continue the discussion here or if you like you could create an RFC in the discussions tab above with your proposed solution so others can give feedback before you code the solution. This is not required, but it might be a good idea. If you want to just dig in and code the solution, I'm happy with that, too. |
sure @rohrig. I've holidays starting from tomorrow, I'll look into it once I'm back. |
The text was updated successfully, but these errors were encountered: