Conversation
Composer package changes
|
|
Coverage report for commit: 6b50ef3 Summary - Lines: - | Methods: -
🤖 comment via lucassabreu/comment-coverage-clover |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
mvdhoek1
left a comment
There was a problem hiding this comment.
Waar zit de switch tussen dev omgevingen en het gebruik van dit component versus productie omgevingen waarbij SeoPress wel de broodkruimels verzorgt? Of gaat het vinkje voor broodkruimels sowieso uit?
src/Components/Breadcrumb.php
Outdated
| return []; | ||
| } | ||
|
|
||
| $parentPageSlug = get_all_post_type_supports(get_post_type($postId))['parent-page'][0]['slug'] ?? null; |
There was a problem hiding this comment.
Iets leesbaarder:
$postType = get_post_type($postId);
$supports = get_all_post_type_supports($postType);
$parentPageSlug = $supports['parent-page'][0]['slug'] ?? null;
De broodkruimels worden nu altijd getoond, er is geen switch. Vóór deze change gebruikten we die van SEOPress, die willen we weg hebben (omdat we die altijd uitzetten op dev) |
Oh SeoPress blijft gewoon draaien maar dan zonder broodkruimels? |
Precies! |
2d38f0c to
b3de77e
Compare
b3de77e to
6b50ef3
Compare
|
|
||
| ### Breadcrumb | ||
|
|
||
| U ses the [Log1x/crumb](https://github.com/Log1x/crumb) package to generate a breadcrumb based on the current page. Also adds posts added through the `parent-page` supports feature. Usage: |
| if ( | ||
| true === $blog && | ||
| get_option('show_on_front') === 'page' && | ||
| ! empty($blogId = get_option('page_for_posts')) && |
There was a problem hiding this comment.
Dit is toch geen blogId maar een postId?
There was a problem hiding this comment.
Eigen breadcrumb component toegevoegd. Omdat SEOPress voor een trage dev omgeving zorgt, hebben we de plugin altijd uitstaan. Gevolg: geen breadcrumbs zichtbaar. Dat is nu opgelost. En meteen weer een afhankelijkheid minder van SEOPress.
Hij gebruikt onderwater Log1x/crumb. De pagina die je toevoegt onder de
parent-pagesupports wordt meegenomen.Gebruik:
Maar je kunt ook je eigen collection meegeven voor als je het nodig hebt op bijv. een custom route:
Eigenlijk doet de Log1x/crumb package niet bijster veel. We kunnen overwegen om die logica hier onder te brengen, zodat we nóg een dependency minder hebben.
I.c.m. deze PR geïmplementeerd in Brave: yardinternet/brave#47 (changes)