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

After renaming formtypes with coreshop[], form elements where not fou… #1791

Merged
merged 1 commit into from
Nov 23, 2021

Conversation

D37R4C7
Copy link
Contributor

@D37R4C7 D37R4C7 commented Nov 23, 2021

…nd by javascript

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Fixed tickets #...

Fixes renaming from commit
8ae0a5f

in shop.js


German description

Hier werden die Form Input anders benannt:
8ae0a5f

$form = $this->formFactory->createNamed('coreshop', AddressType::class, $cart, $options);

Damit heißen die nachher auf der Seite z.B. nicht mehr

sondern

und werden damit nicht mehr von Javascript in der shop.js gefunden

var $invoiceAddress = $addressStep.find('select[name="invoiceAddress"]'),

var $invoiceAddress = $addressStep.find('select[name="invoiceAddress"]'),

@CLAassistant
Copy link

CLAassistant commented Nov 23, 2021

CLA assistant check
All committers have signed the CLA.

@dpfaffenbauer dpfaffenbauer added this to the 2.2.11 milestone Nov 23, 2021
@dpfaffenbauer dpfaffenbauer self-assigned this Nov 23, 2021
@dpfaffenbauer
Copy link
Member

@D37R4C7 please accept the CLA

@dpfaffenbauer dpfaffenbauer merged commit 6174c89 into coreshop:2.2 Nov 23, 2021
@dpfaffenbauer
Copy link
Member

thanks

ramundomario added a commit to ramundomario/CoreShop that referenced this pull request Dec 22, 2021
ramundomario added a commit to ramundomario/CoreShop that referenced this pull request Dec 23, 2021
ramundomario added a commit to ramundomario/CoreShop that referenced this pull request Dec 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants