Repository navigation
fix: correct Encarnación spelling and accent-insensitive address matching - #708
kaio-donadelli wants to merge 5 commits into
Conversation
LOC-28704 — cocacolapy requested the accented city name in Itapúa. Co-authored-by: Cursor <cursoragent@cursor.com>
|
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:
And then you just need to merge your PR when you are ready! There is no need to create a release commit/tag.
|
🛡️ SDD Check — action requiredI couldn't detect an SDD in this PR. Please check one option below (requires write access to the repo):
|
fdaciuk
left a comment
fdaciuk
left a comment
There was a problem hiding this comment.
Duas dúvidas em linha antes de seguir — ver comentários inline.
Use accent-insensitive matching for non-geolocated addresses and add regression test. Co-authored-by: Cursor <cursoragent@cursor.com>
Fix Sonar warning about accentInsensitive controlling normalizeOptionValue behavior. Co-authored-by: Cursor <cursoragent@cursor.com>
fdaciuk
left a comment
fdaciuk
left a comment
There was a problem hiding this comment.
Bug bloqueante em endereços de 3 níveis (ver comentário inline); a descrição do PR também não menciona a mudança de comportamento em validateAddress.ts nem segue o formato de seções do AGENTS.md.
Normalize nested optionsMap keys, resolve SelectPostalCode values for saved addresses, and expand validation/selector test coverage. Co-authored-by: Cursor <cursoragent@cursor.com>
kaio-donadelli
changed the title
Replace Ljubljana placeholder with San Salvador address, keeping each locale prefix. Co-authored-by: Cursor <cursoragent@cursor.com>
|
0 New Issues
5 Fixed Issues
0 Accepted Issues
| "address-form.geolocation.example.SAU": "Ex.: 8078 Al Imam Saud Ibn Abdul Aziz Branch Rd, Al Mursilat, Riyadh 12464", | ||
| "address-form.geolocation.example.SGP": "Ex.: 15-3 Jalan Riang 358987", | ||
| "address-form.geolocation.example.SLV": "Ex.: Štefanova ulica 6, 1000 Ljubljana", | ||
| "address-form.geolocation.example.SLV": "Ex.: Av. Masferrer Norte, San Salvador", |
There was a problem hiding this comment.
Esse ajuste corrigiu o texto certo, mas foi feito à mão aqui e em mais 25 arquivos de idioma (messages/*.json + context.json), além de en.json. O Crowdin ainda é usado para traduzir este repositório? Se sim, essa edição manual pode ser sobrescrita na próxima sincronização, voltando ao texto errado em todo idioma que não seja en. Se o Crowdin não é mais usado, tudo bem manter a edição direta, mas talvez valha atualizar a regra do repositório que diz que só en.json deve ser editado manualmente.
| ) | ||
| } | ||
|
|
||
| export function findOptionsMapKey<T>( |
There was a problem hiding this comment.
findOptionsMapKey é função nova em react/selectors/ e não tem teste próprio — os casos em que não encontra correspondência (options ou value vazios ou sem match) não são testados, só o caminho de sucesso (via getThirdLevelOptions/getThreeLevelsPostalCodes). A regra de testes do repositório exige teste para toda função nova em react/selectors/.
What is the purpose of this pull request?
Fix Paraguay Encarnación spelling while keeping saved addresses and postal code selectors working when accents differ.
Summary
Encarnacion→Encarnacióninreact/country/PRY.tsvalidateAddress.ts(strict for geolocated fields)optionsMapand postal code lookups withcleanStrSelectPostalCodeblank value when saved city spelling differs from country dataTests
validateAddress,fields,postalCode, andSelectPostalCoderegression tests withcountry/__mocks__fixturesTracked in LOC-28704