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
Changes of GR(Greece) PostCode Pattern #36696
Changes of GR(Greece) PostCode Pattern #36696
Conversation
Hi @Vasudev-22. 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. |
@ishakhsuvarov: will this give issues for customers in Magento that already have stored addresses with zip codes in the old format if they want to check out again? Maybe this needs to be tested if it hasn't been yet, before this gets approved? It might be safer to allow both the old format (with space) and the new format (without space), so a regex like: Just my 2 cents, maybe my remark makes no sense... |
@hostep , |
Why? |
Before it shown the postal-code in space format, that was an core issue and this is in wrong format of that country specific pin-code, so we correcting in our system, that's why we need to show them as warning to update it correctly as per your country pin-code. |
I still would like to hear a second opinion on this, @ishakhsuvarov what do you think, or can you drag in somebody who knows a lot about this? |
Sure, I believe you guys, I was just concerned about causing potential problems/annoyances to existing customers of shops if they have entered and stored their postal code in the old (wrong) format. But it seems like nobody agrees here with my remarks, so feel free to continue with this PR 🙂 |
@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 message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@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 message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
It looks like the bot had moved this PR from **Extended testing ** to Ready for testing. Hence, again moving this PR to |
@magento run Functional Tests B2B, Functional Tests CE, Functional Tests EE, Integration Tests, Static Tests |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@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 message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@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 message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
@magento run Functional Tests EE,Functional Tests CE,Functional Tests B2B |
The requested builds are added to the queue. You should be able to see them here within a few minutes. Please message the #magento-devops slack channel if they don't show in a reasonable amount of time and a representative will look into any issues. |
e083a23
into
magento:2.4-develop
Description (*)
Change Pattern for GR PostCodes
Related Pull Requests
Fixes for #36676
Fixed Issues (if relevant)
Manual testing scenarios (*)
Questions or comments
Contribution checklist (*)