Skip to content

Manufacturer 404 links issue#1042

Merged
tblivet merged 1 commit into
PrestaShop:2.xfrom
tblivet:fix/brand-link-404
Jun 11, 2026
Merged

Manufacturer 404 links issue#1042
tblivet merged 1 commit into
PrestaShop:2.xfrom
tblivet:fix/brand-link-404

Conversation

@tblivet

@tblivet tblivet commented Jun 9, 2026

Copy link
Copy Markdown
Contributor
Questions Answers
Description? Fix manufacturer 404 links issue
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? #1030
Sponsor company @PrestaShopCorp
How to test? Follow the issue #1030

@tblivet tblivet added this to the v2.1.0 milestone Jun 9, 2026
@github-project-automation github-project-automation Bot moved this to Ready for review in PR Dashboard Jun 9, 2026
@tblivet tblivet linked an issue Jun 9, 2026 that may be closed by this pull request
2 tasks
@ps-jarvis ps-jarvis added the Waiting for QA Status: Action required, Waiting for test feedback label Jun 9, 2026
@ps-jarvis ps-jarvis moved this from Ready for review to To be tested in PR Dashboard Jun 9, 2026

@ingridusta ingridusta left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @tblivet,

I reproduced the bug, and with your fix, aint no more brands displayed :

Enregistrement.de.l.ecran.2026-06-11.a.16.59.36.mov

Thanks for your contribution to the project Hummingbird ! It's QA approved ✅

@ingridusta ingridusta self-assigned this Jun 11, 2026
@ingridusta ingridusta added QA ✔️ Status: Check done, Code approved and removed Waiting for QA Status: Action required, Waiting for test feedback labels Jun 11, 2026
@ps-jarvis ps-jarvis added the Waiting for QA Status: Action required, Waiting for test feedback label Jun 11, 2026
@tblivet tblivet removed the Waiting for QA Status: Action required, Waiting for test feedback label Jun 11, 2026
@tblivet tblivet merged commit aa8a6d0 into PrestaShop:2.x Jun 11, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from To be tested to Merged in PR Dashboard Jun 11, 2026
@ps-jarvis ps-jarvis moved this from Merged to To be tested in PR Dashboard Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

QA ✔️ Status: Check done, Code approved

Projects

Status: To be tested

Development

Successfully merging this pull request may close these issues.

Manufacturer link 404

4 participants