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

🐛 [BUG] - Pickup points from module "DPD France" (and possibly others) not displayed correctly #431

Closed
hugo-fasone opened this issue Nov 22, 2024 · 3 comments
Labels
4.0.0 bug Something isn't working

Comments

@hugo-fasone
Copy link

Description

The "DPD France" module (https://addons.prestashop.com/fr/transporteurs/19414-dpd-france.html) injects in the DOM its pickup points using the class "delivery-option" on the shipping step of the checkout page. The class "delivery-option" is in the classic theme (https://github.com/PrestaShop/classic-theme/blob/efb5fa7aa2ca2fb856d5f38b720fb6d39f448526/templates/checkout/_partials/steps/shipping.tpl#L44C35-L44C50) but not in Falcon theme, this issue may not be specific to the "DPD France" module, other modules could possibly use this class in their JS for exemple

Since the class "delivery-option" does not exist, the pickup points are injected below the "Continue" button, which is not inuitive at all.

The fix is to add the class "delivery-option" on the delivery options

Node.js version

N/A

php version

N/A

OS and it's version

N/A

Browsers

No response

Required module/theme

theme

Reproduction steps

1. Install DPD France module 
2. Configure it so the "Pickup" delivery option is available
3. Go to the "delivery" step of the checkout and see that the pickup points are not showed correctly

Logs

No response

@hugo-fasone
Copy link
Author

Fixed by #432

@Oksydan
Copy link
Owner

Oksydan commented Nov 23, 2024

Hi @hugo-fasone,

thank you for creating an issue and contribution.
You might be right but modules should relay on js-delivery-option class instead of delivery-option class but I know that not every theme has implemented js- prefixed class properly 😢

@Oksydan Oksydan added the 4.0.0 label Nov 23, 2024
@Oksydan Oksydan moved this to In Progress in Falcon theme version 4.0.0 Nov 23, 2024
@Oksydan Oksydan moved this from In Progress to Done in Falcon theme version 4.0.0 Nov 23, 2024
@Oksydan
Copy link
Owner

Oksydan commented Nov 23, 2024

Changes has been merged into develop.
I am closing this issue 😉. Thank you for your contribution.

@Oksydan Oksydan closed this as completed Nov 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.0.0 bug Something isn't working
Projects
Development

No branches or pull requests

2 participants