Skip to content

[Forwardport] Trim issue on customer confirmation form#51

Merged
gelanivishal merged 462 commits into2.3-developfrom
2.3-develop-PR-port-16595
Jul 20, 2018
Merged

[Forwardport] Trim issue on customer confirmation form#51
gelanivishal merged 462 commits into2.3-developfrom
2.3-develop-PR-port-16595

Conversation

@gelanivishal
Copy link
Copy Markdown
Owner

Original Pull Request

magento#16595

Related to issue: magento#6058

Description

Trim email address by remove leading or trailing space in confirmation form

Fixed Issues (if relevant)

  1. IE11 user login email validation fails if field has leading or trailing space magento/magento2#6058: IE11 user login email validation fails if field has leading or trailing space

Manual testing scenarios

  1. Open customer confirmation page in Firefox or IE browser.
  2. Try to add space before entering an email address in the Email field.
  3. Copy " johndoe@domain.com " and paste in the Email field. It will automatically remove leading or trailing space.

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)

mageprince and others added 30 commits July 9, 2018 23:39
…popup class to let user click deeper than modals and reach to overlay's div in modal-wrapper div
- Add waitForPageLoad to fix flakiness
- Replacing waitForElement with waitForPageLoad.
- Correcting invalid parameterized selectors.
- Adding action group argument type.
- Adding waitForPageLoad to correct a timing issue.
- Adding missing website xml.
…nter-events rule to .modal-… magento#16664

 - Merge Pull Request magento#16664 from mageprince/magento2:2.3-develop-PR-port-15172
 - Merged commits:
   1. 76ae28f
…agento#16670

 - Merge Pull Request magento#16670 from mageprince/magento2:2.3-develop-PR-port-15791
 - Merged commits:
   1. a409944
…name generation magento#16663

 - Merge Pull Request magento#16663 from mageprince/magento2:2.3-develop-PR-port-15144
 - Merged commits:
   1. 2816764
   2. 2d31c1d
   3. e7677e8
   4. ca3c79c
   5. 311f8ca
   6. 1782c54
   7. 94e0e25
   8. 831e58c
   9. 742ad8c
…#16659

 - Merge Pull Request magento#16659 from Coderimus/magento2:fix/forward-port-dev-di-info-duplicated-info
 - Merged commits:
   1. 74adebd
cpartica and others added 28 commits July 18, 2018 13:38
# Conflicts:
#	dev/tests/acceptance/tests/functional/Magento/FunctionalTest/Cms/Section/CmsPagesPageActionsSection.xml
#	dev/tests/acceptance/tests/functional/Magento/FunctionalTest/Store/Data/StoreData.xml
#	dev/tests/acceptance/tests/functional/Magento/FunctionalTest/Store/Data/StoreGroupData.xml
#	dev/tests/acceptance/tests/functional/Magento/FunctionalTest/Store/Section/AdminNewStoreViewActionsSection.xml
#	dev/tests/acceptance/tests/functional/Magento/FunctionalTest/Store/Section/AdminNewWebsiteActionsSection.xml
#	dev/tests/acceptance/tests/functional/Magento/FunctionalTest/Store/Section/AdminStoreGroupActionsSection.xml
…y for Multi-store

-Updated static noNameSpaceSchema Location respectively to match new location
…error-message

[borg] MAGETWO-77744: [Magento Cloud] - Error message when uploading unsupported file format
[earl] MAGETWO-91606: Upgrade to 2.2.2 and above took long time
 - merged latest code from mainline branch
Accepted Public Pull Requests:
 - magento#16909: Stabilize Travis CI integration tests suite (by @ishakhsuvarov)
 - magento#16879: [Port 2.3] Captcha: Added unit test for CheckGuestCheckoutObserver (by @rogyar)
 - magento#16820: Add missing false-check to the ConfiguredRegularPrice price-model (by @ronak2ram)
 - magento/graphql-ce#101: Additional checks for fragments added in category tree (by @rogyar)
 - magento/graphql-ce#102: Fix the category tree depth calculation bug (by @dmytro-ch)
 - magento/graphql-ce#119: GraphQL-116: Wrong category table name resolving (by @naydav)
 - magento#16603: [Forwardport] Fix error in payment void method (by @gelanivishal)
 - magento-engcom/import-export-improvements#109: magento-engcom/import-export-improvements#64: fix issue with Export Type UI (by @dmanners)
 - magento#16596: [Forwardport] Incorrect value NULL was passed to DataObject constructor. It caused � (by @gelanivishal)
 - magento#16518: [Forwardport] Use constant time string comparison in FormKey validator (by @gelanivishal)


Fixed GitHub Issues:
 - magento#100: Oracle and Other RDBMS Status? (reported by @dicgf8) has been fixed in magento/graphql-ce#102 by @dmytro-ch in 2.3-develop branch
   Related commits:
     1. 9116d82

 - magento#116: EE vs CE (reported by @gondo) has been fixed in magento/graphql-ce#119 by @naydav in 2.3-develop branch
   Related commits:
     1. 4a4bbe2

 - magento#16184: Argument 1 passed to Magento\Sales\Model\Order\Payment must be an instance of Magento\Framework\DataObject, none given (reported by @Jakhotiya) has been fixed in magento#16603 by @gelanivishal in 2.3-develop branch
   Related commits:
     1. a1f5fa5
Fixed issues:
- MAGETWO-88432: Filesystem Directory Improvement
[borg] MAGETWO-91528: Customizable options truncated when displaying ordered product in admin
[earl] MAGETWO-67627: CMS Page does not save when same url key with hierarchy for Multi-store
@gelanivishal gelanivishal merged commit 113c2fe into 2.3-develop Jul 20, 2018
gelanivishal pushed a commit that referenced this pull request Aug 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.