-
Notifications
You must be signed in to change notification settings - Fork 9.4k
Fixed issue #22841 Telephone, Company & Fax fields does not display on Customer Registration Page #27608
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
Open
gaurav-473
wants to merge
29
commits into
magento:2.4-develop
Choose a base branch
from
gaurav-473:2.4-develop-22841
base: 2.4-develop
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Fixed issue #22841 Telephone, Company & Fax fields does not display on Customer Registration Page #27608
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
2197d00
Added configuration for showing address fields
gaurav-473 34d04a2
Update register.phtml
gaurav-473 2a76acd
Update config.xml
gaurav-473 f963bc7
fixed static test
gaurav-473 50388ce
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Kilo 5c358b2
MFTF test
engcom-Kilo 76a3463
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Kilo 4eb5022
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Kilo aed35f9
Fix static test
engcom-Kilo 9b0ae83
Correction MFTF test.
engcom-Kilo 64bb6ea
Correct typo.
engcom-Kilo 995e62b
Updated i18n.
engcom-Kilo cb4390c
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Kilo 853df84
Added testCaseId fot MFTF test.
engcom-Kilo 0f5a1c2
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Kilo 0caa710
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 89df850
improve
engcom-Charlie 7cf1e50
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie a82390f
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 40187f5
Merge branch '2.4-develop' into 2.4-develop-22841
4a2516e
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 98488fa
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 083cb2e
fix broken region selector
engcom-Charlie a02fca3
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 78c9c43
fix mftf
engcom-Charlie be93ce9
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 78a7b57
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie 8072400
Merge branch '2.4-develop' into 2.4-develop-22841
engcom-Charlie c78bfb6
Merge branch '2.4-develop' into 2.4-develop-22841
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
32 changes: 32 additions & 0 deletions
32
.../Mftf/ActionGroup/StorefrontCustomerCreateAccountWithAdditionalInformationActionGroup.xml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,32 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
<!-- | ||
/** | ||
* Copyright © Magento, Inc. All rights reserved. | ||
* See COPYING.txt for license details. | ||
*/ | ||
--> | ||
|
||
<actionGroups xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/actionGroupSchema.xsd"> | ||
<actionGroup name="StorefrontCustomerCreateAccountWithAdditionalInformationActionGroup" extends="SignUpNewUserFromStorefrontActionGroup"> | ||
<annotations> | ||
<description>EXTENDS: SignUpNewUserFromStorefrontActionGroup. Fills prefix, suffix, DOB, VAT number, gender, company, telephone, fax, address fields.</description> | ||
</annotations> | ||
<arguments> | ||
<argument name="dob" defaultValue="{{EN_US_DATE.short4DigitYear}}" type="string"/> | ||
<argument name="address" type="entity"/> | ||
<argument name="customerGender" defaultValue="{{Gender.male}}" type="string"/> | ||
</arguments> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.prefixField}}" userInput="{{Customer.prefix}}" after="fillLastName" stepKey="fillPrefixName"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.suffixField}}" userInput="{{Customer.suffix}}" after="fillPrefixName" stepKey="fillSuffixName"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.dobField}}" userInput="{{dob}}" after="fillSuffixName" stepKey="fillDob"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.taxvatField}}" userInput="{{Customer.taxvat}}" after="fillDob" stepKey="fillTaxvat"/> | ||
<selectOption selector="{{StorefrontCustomerCreateFormSection.genderField}}" userInput="{{customerGender}}" after="fillTaxvat" stepKey="fillGender"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.companyField}}" userInput="{{address.company}}" after="fillGender" stepKey="fillCompany"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.telephoneField}}" userInput="{{address.telephone}}" after="fillCompany" stepKey="fillPhoneNumber"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.streetField}}" userInput="{{address.street[0]}}" after="fillPhoneNumber" stepKey="fillStreetAddress"/> | ||
<selectOption selector="{{StorefrontCustomerCreateFormSection.countryField}}" userInput="{{address.country_id}}" after="fillStreetAddress" stepKey="selectCounty"/> | ||
<selectOption selector="{{StorefrontCustomerCreateFormSection.regionField}}" userInput="{{address.state}}" after="selectCounty" stepKey="selectRegion" /> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.cityField}}" userInput="{{address.city}}" after="selectRegion" stepKey="fillCityName"/> | ||
<fillField selector="{{StorefrontCustomerCreateFormSection.postcodeField}}" userInput="{{address.postcode}}" after="fillCityName" stepKey="fillZip"/> | ||
</actionGroup> | ||
</actionGroups> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
56 changes: 56 additions & 0 deletions
56
...Magento/Customer/Test/Mftf/Test/StorefrontCreateCustomerWithAdditionalInformationTest.xml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,56 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
<!-- | ||
/** | ||
* Copyright © Magento, Inc. All rights reserved. | ||
* See COPYING.txt for license details. | ||
*/ | ||
--> | ||
|
||
<tests xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | ||
xsi:noNamespaceSchemaLocation="urn:magento:mftf:Test/etc/testSchema.xsd"> | ||
<test name="StorefrontCreateCustomerWithAdditionalInformationTest"> | ||
<annotations> | ||
<features value="Customer"/> | ||
<stories value="Create a Customer via the Storefront with Additional Information"/> | ||
<title value="Customer should be able to create an account with additional information via the storefront"/> | ||
<description value="Customer should be able to create an account with additional information such like Company, VAT number, Phone via the storefront"/> | ||
<severity value="MAJOR"/> | ||
<testCaseId value="MC-36076"/> | ||
<group value="customer"/> | ||
<group value="create"/> | ||
</annotations> | ||
<before> | ||
<magentoCLI command="config:set customer/address/show_address_fields 1" stepKey="setShowAddressInformation"/> | ||
<magentoCLI command="config:set customer/address/prefix_show opt" stepKey="setShowPrefix"/> | ||
<magentoCLI command="config:set customer/address/middlename_show opt" stepKey="setShowMiddleName"/> | ||
<magentoCLI command="config:set customer/address/suffix_show opt" stepKey="setShowSuffix"/> | ||
<magentoCLI command="config:set customer/address/dob_show opt" stepKey="setShowDOB"/> | ||
<magentoCLI command="config:set customer/address/taxvat_show opt" stepKey="setShowTaxNumber"/> | ||
<magentoCLI command="config:set customer/address/gender_show opt" stepKey="setShowGender"/> | ||
<magentoCLI command="config:set customer/address/telephone_show opt" stepKey="setShowTelephone"/> | ||
</before> | ||
<after> | ||
<magentoCLI command="config:set customer/address/show_address_fields 0" stepKey="setShowAddressInformation"/> | ||
<magentoCLI command="config:set customer/address/prefix_show ''" stepKey="setShowPrefix"/> | ||
<magentoCLI command="config:set customer/address/middlename_show ''" stepKey="setShowMiddleName"/> | ||
<magentoCLI command="config:set customer/address/suffix_show ''" stepKey="setShowSuffix"/> | ||
<magentoCLI command="config:set customer/address/dob_show ''" stepKey="setShowDOB"/> | ||
<magentoCLI command="config:set customer/address/taxvat_show ''" stepKey="setShowTaxNumber"/> | ||
<magentoCLI command="config:set customer/address/gender_show ''" stepKey="setShowGender"/> | ||
<magentoCLI command="config:set customer/address/telephone_show req" stepKey="setShowTelephone"/> | ||
<magentoCLI command="indexer:reindex" stepKey="reindex"/> | ||
|
||
<actionGroup ref="AdminLoginActionGroup" stepKey="login"/> | ||
<actionGroup ref="DeleteCustomerByEmailActionGroup" stepKey="deleteNewUser"> | ||
<argument name="email" value="{{CustomerEntityOne.email}}"/> | ||
</actionGroup> | ||
<actionGroup ref="AdminLogoutActionGroup" stepKey="logout"/> | ||
</after> | ||
|
||
<actionGroup ref="StorefrontCustomerCreateAccountWithAdditionalInformationActionGroup" stepKey="SignUpNewUser"> | ||
<argument name="Customer" value="CustomerEntityOne"/> | ||
<argument name="dob" value="{{EN_US_DATE.short4DigitYear}}"/> | ||
<argument name="address" value="CustomerAddressSimple"/> | ||
</actionGroup> | ||
</test> | ||
</tests> |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you move this method to the separate viewmodel class? It will prevent SVC failures.
More info there https://devdocs.magento.com/guides/v2.4/extension-dev-guide/view-models.html
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah. No, it won't help to resolve SVC failure as it also added a new configuration field.