Skip to content

Updated zip code list for Venezuela.#689

Open
kaio-donadelli wants to merge 2 commits into
4.xfrom
l10n_VEN-update-4.x
Open

Updated zip code list for Venezuela.#689
kaio-donadelli wants to merge 2 commits into
4.xfrom
l10n_VEN-update-4.x

Conversation

@kaio-donadelli
Copy link
Copy Markdown
Contributor

What is the purpose of this pull request?

To update the zip code list for Venezuela. We ignored restrictions we previously had when updating country files so we can test if zip code removal and other major changes have any negative impact on the platform.

Please note Venezuela uses 4 subdivisions to assign zip codes (state -> municipality -> parish -> neighborhood), so municipality and parish were combined in the city field.
What problem is this solving?

Merchants were having issues with missing zip codes from the outdated list.
How should this be manually tested?

These zip codes were present in the old list, but no longer appear in the new list (due to the more granular nature of the new list):

2043, 2045, 2046, 2049, 2210, 2211, 2215, 2217, 2218, 2219, 2220, 2221, 2329, 2331, 2336, 2337, 2339, 2341, 2342, 2343, 2344, 2345, 2346, 2390, 2391, 2392, 2393, 2394, 2395, 2396, 2397, 2398, 2399, 2400, 2401, 2402, 2403, 2404, 2405, 2406, 2407, 2408, 2409, 2410, 2411, 2412, 3117, 3120, 3121, 3300, 5016, 5018, 6004, 6006, 6010, 6011, 6012, 6013, 6015, 6017, 6018, 6019, 6021, 6200, 6202, 6203, 6204, 6300, 6306, 6313, 6500, 6501, 6502, 6503.

You can use them for testing if the update breaks the purchase flow from those locations.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Requires change to documentation, which has been updated accordingly.

Co-authored-by: Cursor <cursoragent@cursor.com>
@vtex-io-ci-cd
Copy link
Copy Markdown
Contributor

vtex-io-ci-cd Bot commented May 15, 2026

Hi! I'm VTEX IO CI/CD Bot and I'll be helping you to publish your app! 🤖

Please select which version do you want to release:

  • Patch (backwards-compatible bug fixes)

  • Minor (backwards-compatible functionality)

  • Major (incompatible API changes)

And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.

  • No thanks, I would rather do it manually 😞

Copy link
Copy Markdown

@Lurian Lurian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Poderia também deixar na descrição do PR um passo a passo de como testar essas mudanças?

Comment thread react/country/VEN.js
valueIn: 'long_name',
types: ['postal_code'],
required: false,
handler: (address) => {
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seria interessante termos testes para essa função de handler

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.

2 participants