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

Fix wrong class naming in Share.vue #136

Merged

Conversation

ghost
Copy link

@ghost ghost commented Oct 21, 2021

Questions Answers
Description? The CSS put into the Share.vue must target only the share modal and not the create modal.
Type? bug fix
BC breaks? no
Deprecations? no
Fixed ticket? Fixes #26349
How to test? See #26349
Possible impacts? Please indicate what parts of the software we need to check to make sure everything is alright.

Copy link

@NeOMakinG NeOMakinG left a comment

Choose a reason for hiding this comment

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

👍

@NeOMakinG
Copy link

CI is broken, I need to fix it before the QA, gonna be done today or tomorrow!

@ghost ghost changed the title Fix wrong class naming into Share.vue Fix wrong class naming in Share.vue Oct 21, 2021
@NeOMakinG NeOMakinG force-pushed the fix-wrong-class-name-share-vue branch from 854ea65 to 8d72427 Compare October 22, 2021 12:11
@hibatallahAouadni
Copy link

Hello @a-pare

Thanks for your PR!
LGTM, QA ✔️

image

Thanks!

@hibatallahAouadni hibatallahAouadni self-assigned this Oct 27, 2021
@Progi1984 Progi1984 added this to the 2.0.2 milestone Oct 27, 2021
@Progi1984 Progi1984 merged commit 54f0b98 into PrestaShop:dev Oct 27, 2021
@Progi1984
Copy link
Contributor

Thanks @a-pare & @hibatallahAouadni

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants