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

Modify category pagination for SEO purposes #9472

Merged
merged 1 commit into from Sep 6, 2018

Conversation

Projects
None yet
4 participants
@jolelievre
Contributor

jolelievre commented Aug 21, 2018

Questions Answers
Branch? develop
Description? Modify category pagination for SEO purposes, Previous hidden on first page, Next hidden on first page, the canonical address integrates the page number when $page > 1
Type? improvement
Category? FO
BC breaks? no
Deprecations? no
Fixed ticket? http://forge.prestashop.com/browse/BOOM-6260
How to test? Go to a category page (change the number of displayed products per page if needed), check the pagination modifs (Previous hidden on first page, Next hidden on first page) and the canonical values

This change is Reviewable

@jolelievre jolelievre added this to the 1.7.5.0 milestone Aug 21, 2018

Show outdated Hide outdated classes/Link.php Outdated
@jolelievre

This comment has been minimized.

Show comment
Hide comment
@jolelievre

jolelievre Aug 30, 2018

Contributor

Fixes #9507

Contributor

jolelievre commented Aug 30, 2018

Fixes #9507

@marionf marionf self-assigned this Sep 6, 2018

@marionf marionf added QA ✔️ and removed waiting for QA labels Sep 6, 2018

@marionf marionf removed their assignment Sep 6, 2018

@jolelievre jolelievre merged commit 53e88bf into PrestaShop:develop Sep 6, 2018

2 checks passed

Codacy/PR Quality Review Up to standards. A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jolelievre jolelievre deleted the jolelievre:BOOM-6260 branch Sep 6, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment