-
Notifications
You must be signed in to change notification settings - Fork 9.3k
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
added partial fix for issue #2617. #3688
added partial fix for issue #2617. #3688
Conversation
…s and the id set before the confirmation mal is sent out.
Thank you for your contribution. Please accept the contributors license agreement so this PR can be further processed. This can be done by clicking the "Details" link next to the "license/cla" check below. |
internal issue MAGETWO-54580 |
- Added message to clarify and prevent future changes.
Fixes for: MAGETWO-56868 Cannot save a product with images for the second time MAGETWO-54355 [Github][PR] Bugfix: Unable to activate search form on phone #3770 MAGETWO-54580 [Github][PR] added partial fix for issue #2617. #3688 MAGETWO-53808 [Github][PR] Fixed post var name for update attributes #4011 MAGETWO-55950 Automate Create new Email Template test MAGETWO-56197 Write functional test for Account sharing settings availability
Hi @whizkid79 We have just noticed that Contributor License Agreement was not signed for this Pull Request. Could you please follow the CLA Assistant link and sign it? Thank you for collaboration. |
Hi @ishakhsuvarov, |
Thanks @whizkid79
This is necessary just once. All future Pull Requests should not require CLA. |
This way double opt in works and the id set before the confirmation mal is sent out.