Merge branch 'develop' into feature/CXIEP-7764

Author: anjana-bl

Diff for: .github/workflows/pr-title-check.yml

@@ -0,0 +1,69 @@
+name: PR Title Check
+
+on:
+  pull_request:
+    types: [opened, edited, synchronize, reopened]
+
+jobs:
+  check-pr-title:
+    name: PR - Check PR title
+    runs-on: ubuntu-latest
+    steps:
+      - name: Validate PR title
+        id: validate
+        env:
+          PR_TITLE: ${{ github.event.pull_request.title }}
+        run: |
+          echo "Checking PR title: ${PR_TITLE}"
+
+          REGEX='^(docs|feat|fix|perf|refactor|style|test|chore)(\([^)]*\))?:\ .+'
+
+          if [[ ! "$PR_TITLE" =~ $REGEX ]]; then
+            echo "❌ PR title is invalid. Example of a valid title: \`feat: Add user authentication\`, \`chore(deps): update dependency babel-loader to v10\`. Allowed prefixes: docs, feat, fix, perf, refactor, style, test, chore"
+            exit 1
+          fi
+
+  comment-on-failure:
+    needs: check-pr-title
+    if: failure()
+    runs-on: ubuntu-latest
+    steps:
+      - name: Post PR comment on failure
+        uses: actions/github-script@v7
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            const issue_number = context.payload.pull_request.number;
+            const owner = context.repo.owner;
+            const repo = context.repo.repo;
+
+            const commentBody = `# 🚨 PR Title Validation Failed 🚨  
+              
+            Your pull request title does not follow the required format. Please update it to match the expected pattern:  
+
+            **Expected format:**  
+            \`<type>: <subject>\`
+
+            ## **Allowed Types**
+            - **docs:** Changes to documentation only
+            - **feat:** New feature work
+            - **fix:** Bug fixes
+            - **perf:** Code improvements for performance
+            - **refactor:** Code changes that are not bug fixes or features
+            - **style:** Code style changes (e.g., whitespace, formatting)
+            - **test:** Adding or updating tests
+            - **chore:** Build, CI, scripts, configs, etc.
+
+            ## **Example of a valid PR title**
+            ✅ \`feat: Add user authentication\`  
+            ✅ \`fix: Resolve checkout bug\`  
+            ✅ \`docs: Update API documentation\`  
+
+            ❌ **Merge is blocked until the PR title is corrected.**`;
+
+            github.rest.issues.createComment({
+              issue_number: issue_number,
+              owner: owner,
+              repo: repo,
+              body: commentBody
+            });

Diff for: feature-libs/asm/occ/adapters/occ-asm.adapter.spec.ts

@@ -158,13 +158,15 @@ describe('OccAsmAdapter', () => {
       result = data;
     });
     const mockReq: TestRequest = httpMock.expectOne((req) => {
-      return req.method === 'GET';
+      return req.method === 'POST';
     });
 
     expect(mockReq.request.params.get('baseSite')).toBe(baseSite);
     expect(mockReq.request.params.get('sort')).toBe(sort);
-    expect(mockReq.request.params.get('query')).toBe(searchQuery);
-    expect(mockReq.request.params.get('customerListId')).toBe(customerListId);
+    expect(mockReq.request.body).toEqual({
+      query: searchQuery,
+      customerListId: customerListId,
+    });
     expect(mockReq.request.params.get('pageSize')).toBe(pageSize + '');
 
     expect(mockReq.cancelled).toBeFalsy();
@@ -196,12 +198,14 @@ describe('OccAsmAdapter', () => {
       result = data;
     });
     const mockReq: TestRequest = httpMock.expectOne((req) => {
-      return req.method === 'GET';
+      return req.method === 'POST';
     });
 
     expect(mockReq.request.params.get('baseSite')).toBe(baseSite);
     expect(mockReq.request.params.get('sort')).toBe(defaultSort);
-    expect(mockReq.request.params.get('query')).toBe(searchQuery);
+    expect(mockReq.request.body).toEqual({
+      query: searchQuery,
+    });
     expect(mockReq.request.params.get('pageSize')).toBe(pageSize + '');
 
     expect(mockReq.cancelled).toBeFalsy();
@@ -228,12 +232,13 @@ describe('OccAsmAdapter', () => {
       result = data;
     });
     const mockReq: TestRequest = httpMock.expectOne((req) => {
-      return req.method === 'GET';
+      return req.method === 'POST';
     });
 
     expect(mockReq.request.params.get('baseSite')).toBe(baseSite);
     expect(mockReq.request.params.get('sort')).toBe(defaultSort);
     expect(mockReq.request.params.get('query')).toBeNull();
+    expect(mockReq.request.body).toEqual({});
     expect(mockReq.request.params.get('pageSize')).toBeNull();
 
     expect(mockReq.cancelled).toBeFalsy();
@@ -265,12 +270,14 @@ describe('OccAsmAdapter', () => {
       result = data;
     });
     const mockReq: TestRequest = httpMock.expectOne((req) => {
-      return req.method === 'GET';
+      return req.method === 'POST';
     });
 
     expect(mockReq.request.params.get('baseSite')).toBe(baseSite);
     expect(mockReq.request.params.get('sort')).toBe(defaultSort);
-    expect(mockReq.request.params.get('query')).toBe(searchQuery);
+    expect(mockReq.request.body).toEqual({
+      query: searchQuery,
+    });
     expect(mockReq.request.params.get('pageSize')).toBe(pageSize + '');
 
     expect(mockReq.cancelled).toBeFalsy();
@@ -299,12 +306,15 @@ describe('OccAsmAdapter', () => {
       result = data;
     });
     const mockReq: TestRequest = httpMock.expectOne((req) => {
-      return req.method === 'GET';
+      return req.method === 'POST';
     });
 
     expect(mockReq.request.params.get('baseSite')).toBe(baseSite);
     expect(mockReq.request.params.get('sort')).toBeNull();
-    expect(mockReq.request.params.get('query')).toBeNull();
+
+    expect(mockReq.request.body).toEqual({
+      customerListId: 'instoreCustomers',
+    });
     expect(mockReq.request.params.get('pageSize')).toBeNull();
 
     expect(mockReq.cancelled).toBeFalsy();

Diff for: feature-libs/asm/occ/adapters/occ-asm.adapter.ts

@@ -32,6 +32,15 @@ import {
 import { Observable } from 'rxjs';
 import { catchError } from 'rxjs/operators';
 
+interface CustomerSearchParams {
+  /**
+   * Supported parameters for searching
+   */
+  query?: string;
+  orderId?: string;
+  customerListId?: string;
+}
+
 @Injectable()
 export class OccAsmAdapter implements AsmAdapter {
   private activeBaseSite: string;
@@ -91,6 +100,8 @@ export class OccAsmAdapter implements AsmAdapter {
       this.activeBaseSite
     );
 
+    const searchBody: CustomerSearchParams = {};
+
     if (options.sort !== undefined) {
       params = params.set('sort', options.sort);
     } else {
@@ -100,10 +111,10 @@ export class OccAsmAdapter implements AsmAdapter {
     }
 
     if (options.query !== undefined) {
-      params = params.set('query', options.query);
+      searchBody.query = options.query;
     }
     if (options.orderId !== undefined) {
-      params = params.set('orderId', options.orderId);
+      searchBody.orderId = options.orderId;
     }
 
     if (options.pageSize !== undefined) {
@@ -115,7 +126,7 @@ export class OccAsmAdapter implements AsmAdapter {
     }
 
     if (options.customerListId !== undefined) {
-      params = params.set('customerListId', options.customerListId);
+      searchBody.customerListId = options.customerListId;
     }
 
     const url = this.occEndpointsService.buildUrl(
@@ -127,12 +138,14 @@ export class OccAsmAdapter implements AsmAdapter {
       }
     );
 
-    return this.http.get<CustomerSearchPage>(url, { headers, params }).pipe(
-      catchError((error) => {
-        throw normalizeHttpError(error, this.logger);
-      }),
-      this.converterService.pipeable(CUSTOMER_SEARCH_PAGE_NORMALIZER)
-    );
+    return this.http
+      .post<CustomerSearchPage>(url, searchBody, { headers, params })
+      .pipe(
+        catchError((error) => {
+          throw normalizeHttpError(error, this.logger);
+        }),
+        this.converterService.pipeable(CUSTOMER_SEARCH_PAGE_NORMALIZER)
+      );
   }
 
   bindCart({ cartId, customerId }: BindCartParams): Observable<unknown> {

Diff for: feature-libs/cart/base/components/cart-shared/cart-item-list-row/cart-item-list-row.component.html

@@ -21,6 +21,7 @@
       <div class="cx-info">
         <div *ngIf="item.product?.name" class="cx-name">
           <a
+            [attr.id]="item.product?.code + '_header'"
             class="cx-link"
             [routerLink]="{ cxRoute: 'product', params: item.product } | cxUrl"
             >{{ item.product?.name }}</a

Diff for: feature-libs/checkout/base/components/guards/checkout-auth.guard.ts

@@ -12,7 +12,7 @@ import {
   AuthService,
   SemanticPathService,
 } from '@spartacus/core';
-import { combineLatest, Observable } from 'rxjs';
+import { Observable, combineLatest } from 'rxjs';
 import { filter, map } from 'rxjs/operators';
 import { CheckoutConfigService } from '../services/checkout-config.service';
 

Diff for: integration-libs/opf/checkout/components/opf-checkout-components.module.ts

@@ -6,6 +6,7 @@
 
 import { NgModule } from '@angular/core';
 import { OpfCheckoutBillingAddressFormModule } from './opf-checkout-billing-address-form/opf-checkout-billing-address-form.module';
+import { OpfCheckoutEmailUpdateModule } from './opf-checkout-email-update/opf-checkout-email-update.module';
 import { OpfCheckoutPaymentAndReviewModule } from './opf-checkout-payment-and-review/opf-checkout-payment-and-review.module';
 import { OpfCheckoutPaymentWrapperModule } from './opf-checkout-payment-wrapper/opf-checkout-payment-wrapper.module';
 import { OpfCheckoutPaymentsModule } from './opf-checkout-payments/opf-checkout-payments.module';
@@ -16,6 +17,7 @@ import { OpfCheckoutPaymentsModule } from './opf-checkout-payments/opf-checkout-
     OpfCheckoutPaymentsModule,
     OpfCheckoutBillingAddressFormModule,
     OpfCheckoutPaymentWrapperModule,
+    OpfCheckoutEmailUpdateModule,
   ],
 })
 export class OpfCheckoutComponentsModule {}

Diff for: integration-libs/opf/checkout/components/opf-checkout-email-update/index.ts

@@ -0,0 +1,8 @@
+/*
+ * SPDX-FileCopyrightText: 2025 SAP Spartacus team <[email protected]>
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+export * from './opf-checkout-email-update.component';
+export * from './opf-checkout-email-update.module';

Diff for: integration-libs/opf/checkout/components/opf-checkout-email-update/opf-checkout-email-update.component.html

@@ -0,0 +1,75 @@
+<form (ngSubmit)="onSubmit()" [formGroup]="checkoutLoginForm">
+  <p class="form-legend">
+    {{ 'formLegend.required' | cxTranslate }}
+  </p>
+  <div class="form-group">
+    <label>
+      <span class="label-content">
+        {{ 'checkoutLogin.emailAddress.label' | cxTranslate }}
+        <ng-template [ngTemplateOutlet]="requiredAsterisk"></ng-template>
+      </span>
+      <input
+        required="true"
+        type="email"
+        name="email"
+        class="form-control"
+        formControlName="email"
+        placeholder="{{
+          'checkoutLogin.emailAddress.placeholder' | cxTranslate
+        }}"
+        [attr.aria-describedby]="'emailError'"
+      />
+
+      <cx-form-errors
+        id="emailError"
+        aria-live="off"
+        [translationParams]="{
+          label: 'checkoutLogin.emailAddress.label' | cxTranslate,
+        }"
+        [control]="checkoutLoginForm.get('email')"
+      ></cx-form-errors>
+    </label>
+  </div>
+
+  <div class="form-group">
+    <label>
+      <span class="label-content">
+        {{ 'checkoutLogin.confirmEmail.label' | cxTranslate }}
+        <ng-template [ngTemplateOutlet]="requiredAsterisk"></ng-template>
+      </span>
+      <input
+        required="true"
+        type="email"
+        name="emailConfirmation"
+        class="form-control"
+        formControlName="emailConfirmation"
+        placeholder="{{
+          'checkoutLogin.confirmEmail.placeholder' | cxTranslate
+        }}"
+        [attr.aria-describedby]="'emailConfirmationError'"
+      />
+
+      <cx-form-errors
+        id="emailConfirmationError"
+        aria-live="off"
+        [translationParams]="{
+          label: 'checkoutLogin.confirmEmail.label' | cxTranslate,
+        }"
+        [control]="checkoutLoginForm.get('emailConfirmation')"
+      ></cx-form-errors>
+    </label>
+  </div>
+
+  <button type="submit" class="btn btn-block btn-primary">
+    {{ 'checkoutLogin.continue' | cxTranslate }}
+  </button>
+</form>
+
+<ng-template #requiredAsterisk>
+  <abbr
+    class="text-decoration-none required-asterisk"
+    aria-hidden="true"
+    title="{{ 'common.required' | cxTranslate }}"
+    >*</abbr
+  >
+</ng-template>