fix: (CXSPA-9504) support Chinese address in address book by espada945 · Pull Request #21272 · SAP/spartacus

espada945 · 2026-03-23T08:32:21Z

Summary

Add cascading city/district dropdowns when China is selected as country
Change "State" label to "Province" for Chinese addresses
Make cellphone required for Chinese addresses
Add district field for Chinese addresses

Test plan

Select China as country → verify city/district dropdowns and Province label
Add complete Chinese address → verify successful save
Switch to non-China country → verify form reverts to default

Add province/city/district fields for Chinese addresses, make cellphone required, and display Province instead of State label.

github-actions · 2026-03-24T03:33:14Z

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

cypress · 2026-03-24T05:06:09Z

spartacus Run #52642

Run Properties: Passed #52642 • 51aba9626f ℹ️: Merge 4ec0863e535e1e111b5afd6c40d198619c442d83 into 96062566bce7cf1b0a2140790069...

Project	`spartacus`
Branch Review	`bugfix/CXSPA-9504`
Run status	`Passed #52642`
Run duration	`04m 11s`
Commit	`51aba9626f ℹ️: Merge 4ec0863e535e1e111b5afd6c40d198619c442d83 into 96062566bce7cf1b0a2140790069...`
Committer	`espada945`
View all properties for this run ↗︎

Test results
Failures	`0`
Flaky	`3`
Pending	`0`
Skipped	`0`
Passing	`101`
View all changes introduced in this branch ↗︎

feature-libs/user/profile/components/address-book/address-form/address-form.component.ts

feature-libs/user/profile/components/address-book/address-form/address-form.component.html

feature-libs/user/profile/components/address-book/address-form/address-form.component.ts

feature-libs/user/profile/components/address-book/address-form/address-form.component.spec.ts

…pport - Use inject() for ChineseAddressService instead of modifying constructor - Rename isChinaAddress to isChineseAddress - Extract HTTP calls to ChineseAddressService with proper error handling and typed responses - Use OccEndpointsService.buildUrl() instead of hardcoded URLs - Reset selectedRegion$/selectedCity$ when switching away from CN - Reorder form fields: Province before City for Chinese address - Refresh cities/districts on language change via LanguageService - Skip address verification for Chinese addresses - Display city/district/region/country names instead of isocodes in address book - Add city and cityDistrict fields to Address model - Request FULL fields for address list API

Disable city dropdown until province is selected and district dropdown until city is selected for Chinese addresses. Show helpful placeholder text ("Please select a province/city first") when disabled.

- Add province/district/placeholder translations for zh and zh_TW - Refresh Chinese placeholders on language switch - Convert cities$/districts$ from observables to arrays with subscriptions - Fix spec tests to match new array properties - Fix prettier formatting in occ-user-address.adapter.ts

…y ternary

github-actions · 2026-03-31T06:01:44Z

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

github-actions · 2026-04-01T02:18:54Z

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

github-actions · 2026-04-01T02:39:47Z

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

…ch original format

…display

feature-libs/user/profile/components/address-book/address-form/address-form.component.ts

feature-libs/user/profile/components/address-book/address-book.component.ts

feature-libs/user/profile/components/address-book/address-form/address-form.component.ts

feature-libs/user/profile/components/address-book/address-form/chinese-address.service.ts

…dapter layer - Add getCities/getDistricts to UserProfileAdapter, OccUserProfileAdapter, UserProfileConnector, UserProfileFacade, and UserProfileService - Add ChineseCity/ChineseDistrict models to user-profile root - Remove ChineseAddressService; component now uses UserProfileFacade - Fix placeholder not updating by adding markForCheck() after translation - Fix city/district not restored when editing existing Chinese address

…ia OccSiteAdapter - Remove getCities/getDistricts from UserProfileFacade/Adapter/Connector chain - Add getCities/getDistricts to UserAddressService delegating to SiteConnector - Add loadCities/loadDistricts to SiteAdapter and SiteConnector - Implement HTTP calls in OccSiteAdapter with dedicated OCC endpoints - Add chineseAddressCities/chineseAddressDistricts endpoint config to default site-context config - Fix address form: use inline cxTranslate for city/district placeholders to avoid async timing gap - Fix address form: initialize town/district form controls as null so ng-select shows placeholder - Fix address form: restore city form control value in edit flow after region cascade reset

…ider, prettier formatting - Add SiteAdapter mock provider to OpfCheckoutBillingAddressFormComponent spec (required after SiteConnector was added as UserAddressService dependency) - Fix prettier formatting in address-form.component.ts - Use address name instead of isocode for country and region in address card

i336588 · 2026-04-10T04:05:14Z

feature-libs/user/profile/components/address-book/address-form/address-form.component.ts

+    }
+  }
+
+  protected updateChinesePlaceholders(): void {}


This empty method is never used.

i336588 · 2026-04-10T05:55:17Z

projects/core/src/user/facade/user-address.service.ts

+  getCities(
+    regionIsocode: string
+  ): Observable<{ isocode?: string; name?: string }[]> {
+    return this.siteConnector.getCities(regionIsocode);


Using NgRx store pattern instead of direct API call would be better for consistency and performance, as getDeliveryCountries() and getRegions() already use.

Performance Issues with Current Implementation

Problem: Multiple API calls for same data
// In address-form.component.ts:194-218
combineLatest([this.selectedRegion$, this.languageService.getActive()])
.pipe(switchMap(([regionIsocode]) => {
return this.userAddressService.getCities(regionIsocode); // 🔴 API call every time!
}))

Every time language changes → new API call
User switches back to previous region → API call again
Multiple address forms on screen → duplicate API calls

Consistency with Existing Architecture

The codebase follows a clear pattern for location data:

Countries → Store (user/store/reducers/delivery-countries.reducer.ts)
Regions → Store (user/store/reducers/regions.reducer.ts)
Cities → Direct API ❌ (breaks pattern)
Districts → Direct API ❌ (breaks pattern)

…ider, prettier formatting - Add SiteAdapter mock provider to OpfCheckoutBillingAddressFormComponent spec - Add loadCities/loadDistricts to MockSiteAdapter in site.connector.spec.ts - Fix prettier formatting in address-form.component.ts - Display country/region name instead of isocode in address book card

fix: (CXSPA-9504) support Chinese address in address book

acf4b32

Add province/city/district fields for Chinese addresses, make cellphone required, and display Province instead of State label.

espada945 requested review from a team as code owners March 23, 2026 08:32

fix: (CXSPA-9504) update existing unit tests for new dependencies

5768599

github-actions bot marked this pull request as draft March 23, 2026 08:50

espada945 and others added 4 commits March 23, 2026 17:09

Merge branch 'develop' into bugfix/CXSPA-9504

1fa348c

fix: add unit tests for China address logic to meet coverage threshold

bb54f99

fix: format spec file with prettier

8a57374

Merge branch 'develop' into bugfix/CXSPA-9504

6055a54

espada945 marked this pull request as ready for review March 24, 2026 03:30

fix: add null fallback for async pipe in ng-select items

01fe95b

github-actions bot marked this pull request as draft March 24, 2026 03:36

espada945 marked this pull request as ready for review March 24, 2026 04:48

scarai-sap reviewed Mar 24, 2026

View reviewed changes

feature-libs/user/profile/components/address-book/address-form/address-form.component.ts Outdated Show resolved Hide resolved

scarai-sap reviewed Mar 24, 2026

View reviewed changes

i336588 reviewed Mar 24, 2026

View reviewed changes

github-actions bot marked this pull request as draft March 26, 2026 08:36

espada945 and others added 5 commits March 26, 2026 17:09

fix: (CXSPA-9504) disable city/district dropdowns until parent selected

a512114

Disable city dropdown until province is selected and district dropdown until city is selected for Chinese addresses. Show helpful placeholder text ("Please select a province/city first") when disabled.

fix: (CXSPA-9504) format address-form component with prettier

cb57b67

Merge branch 'develop' into bugfix/CXSPA-9504

17fb9be

fix: (CXSPA-9504) reduce complexity in getCardContent, fix unnecessar…

a823920

…y ternary

espada945 marked this pull request as ready for review March 31, 2026 05:43

fix: (CXSPA-9504) trigger CI retry

5915982

github-actions bot marked this pull request as draft March 31, 2026 06:11

espada945 marked this pull request as ready for review March 31, 2026 06:42

github-actions bot marked this pull request as draft March 31, 2026 08:19

espada945 marked this pull request as ready for review April 1, 2026 02:18

Merge branch 'develop' into bugfix/CXSPA-9504

91a1228

github-actions bot marked this pull request as draft April 1, 2026 02:19

espada945 marked this pull request as ready for review April 1, 2026 02:22

fix: (CXSPA-9504) use country isocode for address card display to mat…

6a83f92

…ch original format

github-actions bot marked this pull request as draft April 1, 2026 04:43

fix: (CXSPA-9504) use region isocode instead of name in address card …

becc9fe

…display

espada945 marked this pull request as ready for review April 1, 2026 05:17

Merge branch 'develop' into bugfix/CXSPA-9504

b1f7053

github-actions bot marked this pull request as draft April 7, 2026 06:04

espada945 marked this pull request as ready for review April 7, 2026 06:34

Merge branch 'develop' into bugfix/CXSPA-9504

d7248ae

github-actions bot marked this pull request as draft April 7, 2026 07:59

espada945 marked this pull request as ready for review April 7, 2026 08:10

Merge branch 'develop' into bugfix/CXSPA-9504

4ec0863

github-actions bot marked this pull request as draft April 7, 2026 08:38

i53577 marked this pull request as ready for review April 7, 2026 08:40

i336588 reviewed Apr 8, 2026

View reviewed changes

feature-libs/user/profile/components/address-book/address-form/chinese-address.service.ts Outdated Show resolved Hide resolved

espada945 added 2 commits April 9, 2026 15:08

github-actions bot marked this pull request as draft April 9, 2026 07:10

espada945 and others added 3 commits April 9, 2026 15:22

fix: (CXSPA-9504) restore JSDoc comments in SiteAdapter

da98c45

Merge branch 'develop' into bugfix/CXSPA-9504

c90a80b

i336588 reviewed Apr 10, 2026

View reviewed changes

Conversation

espada945 commented Mar 23, 2026

Summary

Test plan

Uh oh!

github-actions bot commented Mar 24, 2026

Merge Checks Failed

Uh oh!

cypress bot commented Mar 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

spartacus Run #52642

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

github-actions bot commented Mar 31, 2026

Merge Checks Failed

Uh oh!

github-actions bot commented Apr 1, 2026

Merge Checks Failed

Uh oh!

github-actions bot commented Apr 1, 2026

Merge Checks Failed

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

i336588 Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

i336588 Apr 10, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

cypress bot commented Mar 24, 2026 •

edited

Loading