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
Issue 36196: Customer attribute and customer address attribute alway … #36197
Conversation
Hi @rogerdz. Thank you for your contribution
❗ Automated tests can be triggered manually with an appropriate comment:
You can find more information about the builds here ℹ️ Run only required test builds during development. Run all test builds before sending your pull request for review. For more details, review the Magento Contributor Guide documentation. 🕙 You can find the schedule on the Magento Community Calendar page. 📞 The triage of Pull Requests happens in the queue order. If you want to speed up the delivery of your contribution, join the Community Contributions Triage session to discuss the appropriate ticket. ✏️ Feel free to post questions/proposals/feedback related to the Community Contributions Triage process to the corresponding Slack Channel |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
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.
While in some cases I would say that it's better to add a default value instead of removing canRestore
, in this case there's core logic that would get broken if the default value was "No", which is a string that evaluates to 1
or true
when cast to integer and boolean respectively. Not to mention any 3rd party logic that could break because of that. So removal of canRestore
is definitely a safer and better option.
Approved. Thank you for the contribution!
To whom it may concern - this PR introduced no data or logic changes to out-of-the-box magento installation (it only affects admins' ability to go back to the initial config - a possibility that in this case never worked anyway) and therefore is not a cause of any test failures. Because of that, failures of integration and functional tests can be disregarded. |
@magento run Integration Tests, Functional Tests CE, Functional Tests EE, Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run Functional Tests CE, Functional Tests EE, Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run Functional Tests CE, Functional Tests EE, Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
Hi @rogerdz , Thanks for your contribution and collaboration. As you explained in last comment PR is removing the
Also actual issue was reported was that -> After changing the configuration to 'required' for all those options and save it .All mentioned field become mandatory and then, if reporter want to change to system default then, it is not happening by clicking on Please provide your input how to fix it by not removing checkbox. Thanks |
…require when select 'Use Default' in config
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
Hi @engcom-Lima, thank you for your response. |
@magento run Functional Tests B2B , Functional Tests CE, Functional Tests EE, Integration Tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@magento run all tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please re-request them if they don't show in a reasonable amount of time. |
@rogerdz Thank you fixing this issue. It will be taken care into one of the Internal JIRA ticket. Once Internal PR will get merged, we will notify. Thank you! |
Hello, As I can see this issue got fixed in the scope of the internal Jira ticket AC-6748 by the internal team Based on the Jira ticket, the target version is 2.4.7-beta1. Thanks |
Description (*)
Related Pull Requests
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)