Skip to content
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

Improved odin_breadcrumbs() with Woocommerce #307

Merged
merged 2 commits into from
Jul 15, 2015
Merged

Improved odin_breadcrumbs() with Woocommerce #307

merged 2 commits into from
Jul 15, 2015

Conversation

AdsonCicilioti
Copy link
Contributor

  • Enabled parent for category crumbs on single and archives/category
    pages.
  • Fix post category parent without <li> tag.
  • Correct name WooCommerce Shop page.

Fix to issue #208

- Enabled parent for category crumbs on single and archives/category
pages.
- Correct name WooCommerce Shop page
@adammacias adammacias added this to the v2.8.0 milestone Jul 10, 2015
@adammacias
Copy link
Member

@AdsonCicilioti puxei aqui para testar, e tá dando errado aqui 😭

Olha só...

happyninjamework

Lembrando que, o primeiro breadcrump é do Odin, e o segundo, WooCommerce.

Esse simple aí é categoria de outro produto. 😮

e o link do simple está assim:

/?taxonomy=product_type&term=simple - está listando produtos que não estão nessa categoria.

quando eu acho que deveria ser:

/?product_cat=simple - lista apenas um produto, o dono da categoria.

Acho que o esquema é "clonar" o breadcrump do WC pro Odin 😁

@AdsonCicilioti
Copy link
Contributor Author

Vixi.. É mesmo. Vou dar uma olhada assim que chegar em casa.

Eu analizei o do WC, realmente funfa legal para todas páginas. Porém teríamos que ajustar o output do HTML pq o padrão do WC é feio. XD.. Mas valew mano!

@AdsonCicilioti
Copy link
Contributor Author

Cara comi mosca 😆 .. Esqueci de aplicar a variável $taxy. Assim que puder testa de novo @adammacias .

Na verdade man, aquele simple é um dos termos da taxonomia product_type que gerencia se o produto é simples, variável, externo ou grupo. É a taxonomia padrão dos produtos WC. O que fiz foi ajustar ele pra pegar os temos da taxonomia product_cat.

@adammacias
Copy link
Member

haha, saquei! Sem problemas, quando puder, me avisa pra eu testar novamente.

PS. desfaz esses últimos commits que não tem relação com o PR em questão ___

@adammacias
Copy link
Member

Ah, agora que eu vi que tu "reverteu" com outro commit haha,
@AdsonCicilioti Tem um jeito mais fácil e certo de fazer isso, com git reset.

Desfazendo os 2 últimos commits

git reset --hard HEAD~2

@AdsonCicilioti
Copy link
Contributor Author

Pow valew man.. Sou nub em git ainda ha ha! Se eu mandar esse reset agora vai de boa?

Foi de boa, porém não tava atualizando a repo com git push, daí mandei um git push --force.

Pode testar agora @adammacias

@adammacias adammacias merged commit cb97f2c into wpbrasil:master Jul 15, 2015
@adammacias
Copy link
Member

Agora tá redondinho! valeu @AdsonCicilioti 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants