feat(carousel): introduce new catalog group layout options and update related components for enhanced product display
This commit is contained in:
@@ -65,6 +65,7 @@ export interface CatalogItemDetail {
|
||||
}
|
||||
|
||||
export type CatalogProductLayout = 'row' | 'column_with_image' | 'column_with_cart';
|
||||
export type CatalogGroupLayout = 'paginated' | 'simple' | 'simple_vertical' | 'carousel';
|
||||
|
||||
export interface CatalogFeaturedItemVariant {
|
||||
id: number;
|
||||
@@ -82,10 +83,14 @@ export interface CatalogFeaturedItem {
|
||||
variants?: CatalogFeaturedItemVariant[];
|
||||
}
|
||||
|
||||
export type CatalogFeaturedItems =
|
||||
ApiPaginatedResponse<CatalogFeaturedItem[]> | CatalogFeaturedItem[];
|
||||
|
||||
export interface CatalogFeaturedGroup {
|
||||
id: number;
|
||||
title: string;
|
||||
layout: CatalogProductLayout;
|
||||
group_layout: CatalogGroupLayout;
|
||||
group_order: number;
|
||||
items: ApiPaginatedResponse<CatalogFeaturedItem[]>;
|
||||
items: CatalogFeaturedItems;
|
||||
}
|
||||
|
||||
@@ -8,7 +8,7 @@ import { ApiResponse } from '../api-response.interface';
|
||||
import { TenantService } from '../tenant.service';
|
||||
import {
|
||||
CatalogFeaturedGroup,
|
||||
CatalogFeaturedItem,
|
||||
CatalogFeaturedItems,
|
||||
CatalogItemDetail,
|
||||
Product,
|
||||
} from './catalog.interface';
|
||||
@@ -39,8 +39,8 @@ export class CatalogService {
|
||||
getFeaturedGroupItems(
|
||||
featuredGroupId: number,
|
||||
params?: ApiPaginationQueryParams,
|
||||
): Observable<ApiPaginatedResponse<CatalogFeaturedItem[]>> {
|
||||
return this.http.get<ApiPaginatedResponse<CatalogFeaturedItem[]>>(
|
||||
): Observable<CatalogFeaturedItems> {
|
||||
return this.http.get<CatalogFeaturedItems>(
|
||||
`${this.tenantApiUrl}/catalog/featured-groups/${featuredGroupId}/items`,
|
||||
{ params: this.buildHttpParams(params) },
|
||||
);
|
||||
|
||||
@@ -56,6 +56,7 @@ function createCatalog(
|
||||
id: 7,
|
||||
title: 'Destacados',
|
||||
layout: 'column_with_image',
|
||||
group_layout: 'paginated',
|
||||
group_order: 0,
|
||||
items: createItemsPage(items, currentPage, lastPage),
|
||||
},
|
||||
|
||||
@@ -53,8 +53,7 @@ export class StoreHomePageComponent implements OnInit, OnDestroy {
|
||||
|
||||
ngOnInit(): void {
|
||||
const resolvedData = this.route.snapshot.data['catalogData'] as
|
||||
| StoreHomeCatalogResolvedData
|
||||
| undefined;
|
||||
StoreHomeCatalogResolvedData | undefined;
|
||||
|
||||
if (resolvedData) {
|
||||
this.applyResolvedData(resolvedData);
|
||||
@@ -78,6 +77,7 @@ export class StoreHomePageComponent implements OnInit, OnDestroy {
|
||||
|
||||
if (
|
||||
!group ||
|
||||
Array.isArray(group.items) ||
|
||||
page < 1 ||
|
||||
page === group.items.meta.current_page ||
|
||||
this.isGroupLoading(groupId)
|
||||
@@ -112,18 +112,15 @@ export class StoreHomePageComponent implements OnInit, OnDestroy {
|
||||
}
|
||||
|
||||
protected onAddToCart(event: ProductListCartEvent): void {
|
||||
this.cartService
|
||||
.addItem(event.product.id, event.variant ?? null, event.quantity)
|
||||
.subscribe({
|
||||
next: (response) => {
|
||||
this.toastService.success(response.message || 'Producto agregado al carrito');
|
||||
},
|
||||
error: (error: HttpErrorResponse) => {
|
||||
const message =
|
||||
error.error?.message || 'No se pudo agregar el producto al carrito.';
|
||||
this.toastService.danger(message);
|
||||
},
|
||||
});
|
||||
this.cartService.addItem(event.product.id, event.variant ?? null, event.quantity).subscribe({
|
||||
next: (response) => {
|
||||
this.toastService.success(response.message || 'Producto agregado al carrito');
|
||||
},
|
||||
error: (error: HttpErrorResponse) => {
|
||||
const message = error.error?.message || 'No se pudo agregar el producto al carrito.';
|
||||
this.toastService.danger(message);
|
||||
},
|
||||
});
|
||||
}
|
||||
|
||||
private loadCatalog(): void {
|
||||
|
||||
@@ -3,7 +3,7 @@
|
||||
[class.product-list--row]="effectiveLayout() === 'row'"
|
||||
[class.product-list--column]="effectiveLayout() !== 'row'"
|
||||
>
|
||||
@for (item of items().data; track item.id; let index = $index) {
|
||||
@for (item of itemData(); track item.id; let index = $index) {
|
||||
<div class="product-list__item">
|
||||
@switch (effectiveLayout()) {
|
||||
@case ('row') {
|
||||
@@ -39,13 +39,15 @@
|
||||
}
|
||||
</div>
|
||||
|
||||
@if (items().meta.last_page > 1) {
|
||||
<div class="product-list__paginator d-flex justify-content-center">
|
||||
<app-paginator
|
||||
[page]="items().meta.current_page"
|
||||
[totalPages]="items().meta.last_page"
|
||||
[disabled]="loading()"
|
||||
(pageChange)="pageChange.emit($event)"
|
||||
/>
|
||||
</div>
|
||||
@if (pagination(); as paginatedItems) {
|
||||
@if (paginatedItems.meta.last_page > 1) {
|
||||
<div class="product-list__paginator d-flex justify-content-center">
|
||||
<app-paginator
|
||||
[page]="paginatedItems.meta.current_page"
|
||||
[totalPages]="paginatedItems.meta.last_page"
|
||||
[disabled]="loading()"
|
||||
(pageChange)="pageChange.emit($event)"
|
||||
/>
|
||||
</div>
|
||||
}
|
||||
}
|
||||
|
||||
@@ -3,6 +3,7 @@ import { BrowserTestingModule, platformBrowserTesting } from '@angular/platform-
|
||||
import { afterEach, beforeAll, describe, expect, it, vi } from 'vitest';
|
||||
|
||||
import { ApiPaginatedResponse } from '../../../core/services/api-paginated-response.interface';
|
||||
import { CatalogFeaturedItems } from '../../../core/services/catalog/catalog.interface';
|
||||
import { ProductListComponent, ProductListItem, ProductListLayout } from './product-list.component';
|
||||
|
||||
describe('ProductListComponent', () => {
|
||||
@@ -38,11 +39,14 @@ describe('ProductListComponent', () => {
|
||||
TestBed.resetTestingModule();
|
||||
});
|
||||
|
||||
async function render(layout: ProductListLayout) {
|
||||
async function render(
|
||||
layout: ProductListLayout,
|
||||
productItems: CatalogFeaturedItems = paginatedItems(),
|
||||
) {
|
||||
await TestBed.configureTestingModule({ imports: [ProductListComponent] }).compileComponents();
|
||||
const fixture = TestBed.createComponent(ProductListComponent);
|
||||
fixture.componentRef.setInput('layout', layout);
|
||||
fixture.componentRef.setInput('items', paginatedItems());
|
||||
fixture.componentRef.setInput('items', productItems);
|
||||
fixture.detectChanges();
|
||||
|
||||
return fixture;
|
||||
@@ -128,4 +132,12 @@ describe('ProductListComponent', () => {
|
||||
|
||||
expect(pageChangeSpy).toHaveBeenCalledWith(2);
|
||||
});
|
||||
|
||||
it('renders plain items without pagination', async () => {
|
||||
const fixture = await render('row', items);
|
||||
const element = fixture.nativeElement as HTMLElement;
|
||||
|
||||
expect(element.querySelectorAll('app-product-row-card')).toHaveLength(2);
|
||||
expect(element.querySelector('app-paginator')).toBeNull();
|
||||
});
|
||||
});
|
||||
|
||||
@@ -10,9 +10,9 @@ import {
|
||||
signal,
|
||||
} from '@angular/core';
|
||||
|
||||
import { ApiPaginatedResponse } from '../../../core/services/api-paginated-response.interface';
|
||||
import {
|
||||
CatalogFeaturedItem,
|
||||
CatalogFeaturedItems,
|
||||
CatalogFeaturedItemVariant,
|
||||
CatalogProductLayout,
|
||||
} from '../../../core/services/catalog/catalog.interface';
|
||||
@@ -51,7 +51,7 @@ export class ProductListComponent {
|
||||
private readonly mobile = signal(false);
|
||||
|
||||
readonly layout = input.required<ProductListLayout>();
|
||||
readonly items = input.required<ApiPaginatedResponse<ProductListItem[]>>();
|
||||
readonly items = input.required<CatalogFeaturedItems>();
|
||||
readonly loading = input(false);
|
||||
|
||||
readonly buy = output<ProductListItem>();
|
||||
@@ -61,6 +61,16 @@ export class ProductListComponent {
|
||||
protected readonly effectiveLayout = computed<ProductListLayout>(() =>
|
||||
this.mobile() && this.layout() === 'row' ? 'column_with_cart' : this.layout(),
|
||||
);
|
||||
protected readonly itemData = computed(() => {
|
||||
const items = this.items();
|
||||
|
||||
return Array.isArray(items) ? items : items.data;
|
||||
});
|
||||
protected readonly pagination = computed(() => {
|
||||
const items = this.items();
|
||||
|
||||
return Array.isArray(items) ? null : items;
|
||||
});
|
||||
|
||||
constructor() {
|
||||
afterNextRender(() => {
|
||||
|
||||
Reference in New Issue
Block a user