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

Add shop details support to classic #26832

Closed
wants to merge 6 commits into from
Closed

Conversation

Hlavtox
Copy link
Contributor

@Hlavtox Hlavtox commented Dec 2, 2021

Questions Answers
Branch? develop
Description? Legal information is not displayed on contact page, while there is a field for that in Prestashop settings.
Type? improvement
Category? FO
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #26809
How to test? Fill the legal info in BO, apply PrestaShop/ps_contactinfo#43 and go to contact page.
Possible impacts? None.

This change is Reviewable

@Hlavtox Hlavtox requested a review from a team as a code owner December 2, 2021 09:45
@prestonBot prestonBot added develop Branch Improvement Type: Improvement labels Dec 2, 2021
@prestonBot
Copy link
Collaborator

prestonBot commented Dec 2, 2021

This pull request seems to contain new translation strings. I have summarized them below to ease up review:

(Note: this is an automated message, but answering it will reach a real human)

@prestonBot prestonBot added the Waiting for wording Status: action required, waiting for wording label Dec 2, 2021
NeOMakinG
NeOMakinG previously approved these changes Dec 2, 2021
@Julievrz
Copy link
Contributor

Julievrz commented Dec 2, 2021

Hi @Hlavtox

I've tried to test the PR to better understand the context of the wording. But the registration number doesn't appear on the front office with your PR. Is it just me?

image
image

@Hlavtox
Copy link
Contributor Author

Hlavtox commented Dec 2, 2021

@Julievrz Hi, you must also apply PR from PrestaShop/ps_contactinfo#43.

@Julievrz
Copy link
Contributor

Julievrz commented Dec 2, 2021

Thanks!
So instead of having "Details:" what about having "Registration number:" like in the back office?

@matks matks added the Waiting for author Status: action required, waiting for author feedback label Dec 2, 2021
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Dec 2, 2021

@Julievrz It can't be called "Registration number". For example, in Czech republic, you are required by the law to specify a lot more information, and Contact page is the best way to do it. We should also make line breaks working in this input, or just make it a free HTML.

On top of company name, address, phone, email also:

  • Registration number: 12345678
  • Registration information: The company is registered in the Commercial Register kept at the Regional Court in Prague in section C, insert 32726.
  • VAT number: CZ12345678

@Julievrz
Copy link
Contributor

@Julievrz It can't be called "Registration number". For example, in Czech republic, you are required by the law to specify a lot more information, and Contact page is the best way to do it. We should also make line breaks working in this input, or just make it a free HTML.

On top of company name, address, phone, email also:

  • Registration number: 12345678
  • Registration information: The company is registered in the Commercial Register kept at the Regional Court in Prague in section C, insert 32726.
  • VAT number: CZ12345678

Ok, I understand your point. However, when checking PrestaShop websites, it seems that merchants tend to use the "Legal notice" page to display these details.

https://www.lechocolatdesfrancais.fr/en/
image
https://www.poupadou.com/en/content/legal-notice-2
image
https://www.pylones.com/en/content/5-legal-notices
image

Anyway, I'm not against keeping "Details:". It's quite generic wording and merchants will still be able to modify it according to their needs. I'm just not sure to understand the difference between this and the legal notice. :-)

@Hlavtox
Copy link
Contributor Author

Hlavtox commented Dec 18, 2021

@Julievrz That field was definitely meant to contain more info. It's multiline. It's even written in the helpbox ("Registration information (company ID, VAT number)".

reg

I think that line break support or ability to enter HTML should be provided in a separate PR.


Also, I randomly checked 4 stores I know from memory, where I buy stuff on regular basis. One of the best stores in their field. All have VAT number and registration info on their contact page. 👍

@Julievrz
Copy link
Contributor

@Julievrz That field was definitely meant to contain more info. It's multiline. It's even written in the helpbox ("Registration information (company ID, VAT number)".

reg

I think that line break support or ability to enter HTML should be provided in a separate PR.

Also, I randomly checked 4 stores I know from memory, where I buy stuff on regular basis. One of the best stores in their field. All have VAT number and registration info on their contact page. 👍

Yes, I totally understand! I think the label in the BO is quite confusing though. We can go for "Company details:" if it suits you. ☺️

Co-authored-by: Julie Varisellaz <70583503+Julievrz@users.noreply.github.com>
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Dec 21, 2021

@Julievrz Wording applied, I also changed it in BO, can you check? :-)

@Hlavtox
Copy link
Contributor Author

Hlavtox commented Dec 21, 2021

Also, added support for line breaks.

Co-authored-by: Julie Varisellaz <70583503+Julievrz@users.noreply.github.com>
@Julievrz Julievrz added Wording ✔️ Status: check done, wording approved and removed Waiting for wording Status: action required, waiting for wording labels Dec 21, 2021
@matks matks removed the Waiting for author Status: action required, waiting for author feedback label Dec 23, 2021
matks
matks previously approved these changes Dec 23, 2021
@prestonBot prestonBot added Waiting for wording Status: action required, waiting for wording and removed Wording ✔️ Status: check done, wording approved labels Dec 23, 2021
@Julievrz Julievrz added Wording ✔️ Status: check done, wording approved and removed Waiting for wording Status: action required, waiting for wording labels Dec 23, 2021
@Progi1984 Progi1984 added the Waiting for QA Status: action required, waiting for test feedback label Dec 27, 2021
@khouloudbelguith khouloudbelguith self-assigned this Dec 27, 2021
@khouloudbelguith
Copy link
Contributor

Hi @Hlavtox,

I tested this PR using this PR.
The wording is ok in the BO
image
In the FO > in the footer => is not displayed
image

Thanks!

@khouloudbelguith khouloudbelguith added Waiting for author Status: action required, waiting for author feedback Waiting for dev Status: action required, waiting for tech feedback and removed Waiting for QA Status: action required, waiting for test feedback labels Dec 27, 2021
@Hlavtox
Copy link
Contributor Author

Hlavtox commented Jan 5, 2022

@khouloudbelguith I didn't implement it, I don't think it's needed there...

@PierreRambaud PierreRambaud added Waiting for QA Status: action required, waiting for test feedback and removed Waiting for author Status: action required, waiting for author feedback Waiting for dev Status: action required, waiting for tech feedback labels Jan 19, 2022
Copy link
Contributor

@khouloudbelguith khouloudbelguith left a comment

Choose a reason for hiding this comment

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

Hi @Hlavtox,

In fact, as you can see in this comment
The registration number should be displayed in the contact page but also in the footer, under the email.
This behavior is validated by the product team.

Thank you!

@khouloudbelguith khouloudbelguith added Waiting for author Status: action required, waiting for author feedback and removed Waiting for QA Status: action required, waiting for test feedback labels Jan 24, 2022
@Progi1984
Copy link
Contributor

Hi @Hlavtox,
Since we had no news from you for more than 30 days, I'll close this pull request. Feel free to reopen or open another one if you think it's still relevant.
Thanks!

@Progi1984 Progi1984 closed this Mar 7, 2022
@Hlavtox Hlavtox deleted the patch-5 branch November 21, 2022 09:38
@fabiocranchi
Copy link

Solution:

Edit the file /modules/ps_contactinfo/ps_contactinfo.php
In array $contact_infos, add this line: 'regNumber' => Configuration::get('PS_SHOP_DETAILS'),
In the file just below the line 'company' => Configuration::get('PS_SHOP_NAME'),
Add: 'regNumber' => Configuration::get('PS_SHOP_DETAILS'),

Then, inside the theme, search for ps_contactinfo.tpl and add "
{$contact_infos.regNumber}" without quotes just below "{$contact_infos.address.formatted nofilter}"

tpl file location in classic theme:
themes/classic/modules/ps_contactinfo

Hope it helped =)

@fabiocranchi
Copy link

Hi @Hlavtox,

I tested this PR using this PR. The wording is ok in the BO image In the FO > in the footer => is not displayed image

Thanks!

Solution:

Edit the file /modules/ps_contactinfo/ps_contactinfo.php
In array $contact_infos, add this line: 'regNumber' => Configuration::get('PS_SHOP_DETAILS'),
In the file just below the line 'company' => Configuration::get('PS_SHOP_NAME'),
Add: 'regNumber' => Configuration::get('PS_SHOP_DETAILS'),

Then, inside the theme, search for ps_contactinfo.tpl and add "
{$contact_infos.regNumber}" without quotes just below "{$contact_infos.address.formatted nofilter}"

tpl file location in classic theme:
themes/classic/modules/ps_contactinfo

Hope it helped =)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
develop Branch Improvement Type: Improvement Waiting for author Status: action required, waiting for author feedback Wording ✔️ Status: check done, wording approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Legal info is not displayed in contact module
10 participants