Skip to content

spec: per-geolocation-ubigeo-postal-code - #706

Closed
kaio-donadelli wants to merge 5 commits into
mainfrom
spec/per-geolocation-ubigeo-postal-code
Closed

spec: per-geolocation-ubigeo-postal-code#706
kaio-donadelli wants to merge 5 commits into
mainfrom
spec/per-geolocation-ubigeo-postal-code

Conversation

@kaio-donadelli

@kaio-donadelli kaio-donadelli commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

What is the purpose of this pull request?

Define how Peru geolocation must always resolve INEI ubigeos and never leak Google's 5-digit CPN into checkout addresses (LOC-21884 / Zendesk #1290373).

Summary

  • Spec keeps 6-digit ubigeo as the canonical Peru "postal code" (guidance from CXs)
  • Stops mapping Google postal_code; derives ubigeo from department/province/district with prefix/accent-tolerant matching
  • Includes refreshing Peru country data from INEI 2025; out of scope: switching to CPN, tightening regex to /^\d{6}$/

Co-authored-by: Cursor <cursoragent@cursor.com>
@kaio-donadelli
kaio-donadelli requested a review from a team as a code owner July 30, 2026 17:16
@vtex-io-ci-cd

vtex-io-ci-cd Bot commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

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 😞

kaio-donadelli and others added 4 commits July 30, 2026 15:06
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@sonar-workflows

Copy link
Copy Markdown

Failed Quality Gate failed

  • 7 New Issues (is greater than 0)
  • 0.00% Coverage on New Code (is less than 75.00%)

Project ID: address-form

View in SonarQube

@kaio-donadelli

Copy link
Copy Markdown
Contributor Author

Superseded: the SDD now lives in the implementation PR #707 (specs/per-geolocation-ubigeo-postal-code.md).

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.

1 participant