PrestaShop
PrestaShop copied to clipboard
Remove default meta description
Questions | Answers |
---|---|
Branch? | develop |
Description? | Remove default meta description "Shop powered by PrestaShop", and improve prestashop beginner seo |
Type? | improvement |
Category? | FO |
BC breaks? | no |
Deprecations? | no |
Fixed ticket? | Fixes https://github.com/PrestaShop/PrestaShop/issues/28941 |
How to test? | Install prestashop, see that the meta description is not here anymore |
Possible impacts? | none |
Hi @jf-viguier, nice PR!
I think you can also remove this wording here: https://github.com/PrestaShop/PrestaShop/blob/6f8365ebba40b37e611525597ea0d3f2c907dc76/classes/lang/KeysReference/MetaLang.php#L57
@florine2623
I don't think it's related. We need to export .xlf files for a new release, and then it should be all good.
@jf-viguier could you rebase your PR? :)
This pull request seems to contain new translation strings. I have summarized them below to ease up review:
-
Shop.Navigation
- [ ]
Manufacturers
- [ ]
Brand list
- [ ]
On-sale products
- [ ]
manufacturers
- [ ]
supplier
- [ ]
(Note: this is an automated message, but answering it will reach a real human)
@kpodemski rebase done
@matks rebase done
Closed/reopened to trigger the CI
I think @micka-fdz's comment is not here anymore?
Hi @jf-viguier, nice PR!
I think you can also remove this wording here:
https://github.com/PrestaShop/PrestaShop/blob/6f8365ebba40b37e611525597ea0d3f2c907dc76/classes/lang/KeysReference/MetaLang.php#L57
@atomiix it's done
QA approved, well done! Message to the maintainers: do not forget to milestone it before the merge.
Thanks @jf-viguier