feat(contact): enhance contact page to support dynamic social media links and improve data structure

This commit is contained in:
2026-09-30 15:03:30 -03:00
parent b6ab36437b
commit 1edbd84be6
3 changed files with 212 additions and 93 deletions

View File

@@ -1,15 +1,27 @@
<app-menu-content-section menuCode="help.contact" titleId="contact-page-title"> <app-menu-content-section menuCode="help.contact" titleId="contact-page-title">
<div class="contact-page"> <div class="contact-page">
<div class="contact-page__details"> <div class="contact-page__details">
@if (contact().whatsappUrl && contact().whatsappLabel) { @for (whatsapp of contact().whatsapp; track whatsapp.code) {
<a <a
class="contact-page__contact-link" class="contact-page__contact-link"
[href]="contact().whatsappUrl" [href]="whatsapp.url"
target="_blank" target="_blank"
rel="noopener noreferrer" rel="noopener noreferrer"
> >
<i class="fa-brands fa-whatsapp" aria-hidden="true"></i> <i [class]="whatsapp.icon" aria-hidden="true"></i>
<span>{{ contact().whatsappLabel }}</span> <span>{{ whatsapp.name }}</span>
</a>
}
@for (social of contact().socialMedia; track social.code) {
<a
class="contact-page__contact-link"
[href]="social.url"
target="_blank"
rel="noopener noreferrer"
>
<i [class]="social.icon" aria-hidden="true"></i>
<span>{{ social.name }}</span>
</a> </a>
} }
@@ -35,7 +47,12 @@
</article> </article>
} }
@if (!contact().whatsappUrl && contact().phones.length === 0 && contact().locations.length === 0) { @if (
contact().whatsapp.length === 0 &&
contact().socialMedia.length === 0 &&
contact().phones.length === 0 &&
contact().locations.length === 0
) {
<p class="contact-page__empty">No hay datos de contacto disponibles.</p> <p class="contact-page__empty">No hay datos de contacto disponibles.</p>
} }
</div> </div>

View File

@@ -19,14 +19,55 @@ const tenant: Tenant = {
header_logo: '', header_logo: '',
footer_logo: '', footer_logo: '',
categories: [], categories: [],
social_media: [
{
code: 'whatsapp',
name: 'WhatsApp',
icon: 'fa-brands fa-whatsapp',
url: 'https://wa.me/543412602222',
},
{
code: 'instagram',
name: 'Instagram',
icon: 'fa-brands fa-instagram',
url: 'https://www.instagram.com/comercio',
},
],
phone_numbers: [ phone_numbers: [
{ id: 1, label: 'Principal', number: '+54 9 (0341) 6658247', tel_url: 'tel:+54903416658247', is_main: true }, {
id: 1,
label: 'Principal',
number: '+54 9 (0341) 6658247',
tel_url: 'tel:+54903416658247',
is_main: true,
},
{ id: 2, label: 'Ventas', number: '3411234567', tel_url: 'tel:3411234567', is_main: false }, { id: 2, label: 'Ventas', number: '3411234567', tel_url: 'tel:3411234567', is_main: false },
], ],
addresses: [ addresses: [
{ id: 1, label: 'Gigante de Arroyito', address_text: 'Av. Génova 640, Rosario', latitude: -32.913997, longitude: -60.674567, is_main: true }, {
{ id: 2, label: 'Telepagos', address_text: 'Rioja 1150, piso 12, dpto. 3, Rosario', latitude: -32.94682, longitude: -60.63932, is_main: false }, id: 1,
{ id: 3, label: 'San Martín 2450', address_text: 'San Martín 2450, Santa Fe', latitude: -31.63333, longitude: -60.7, is_main: false }, label: 'Gigante de Arroyito',
address_text: 'Av. Génova 640, Rosario',
latitude: -32.913997,
longitude: -60.674567,
is_main: true,
},
{
id: 2,
label: 'Telepagos',
address_text: 'Rioja 1150, piso 12, dpto. 3, Rosario',
latitude: -32.94682,
longitude: -60.63932,
is_main: false,
},
{
id: 3,
label: 'San Martín 2450',
address_text: 'San Martín 2450, Santa Fe',
latitude: -31.63333,
longitude: -60.7,
is_main: false,
},
], ],
menues: [ menues: [
{ {
@@ -42,42 +83,8 @@ const tenant: Tenant = {
code: 'help.contact', code: 'help.contact',
label: 'Contacto', label: 'Contacto',
parent_menu_code: 'help', parent_menu_code: 'help',
content_type: 'static', content_type: 'dynamic',
route: '/ayuda/contacto', route: '/ayuda/contacto',
static_content: {
whatsapp: {
whatsapp_url: 'https://wa.me/543412602222',
whatsapp_label: 'Chatea con nosotros',
},
phone: '+54 9 (0341) 6658247',
locations: {
rosario: {
label: 'Rosario',
addresses: [
{
label: 'Gigante de Arroyito',
address: 'Av. Génova 640, Rosario',
coordinates: [-32.913997, -60.674567],
},
{
label: 'Telepagos',
address: 'Rioja 1150, piso 12, dpto. 3, Rosario',
coordinates: [-32.94682, -60.63932],
},
],
},
santa_fe: {
label: 'Santa Fe',
addresses: [
{
label: 'San Martín 2450',
address: 'San Martín 2450, Santa Fe',
coordinates: [-31.63333, -60.7],
},
],
},
},
},
submenues: [], submenues: [],
}, },
], ],
@@ -115,7 +122,10 @@ describe('ContactPage', () => {
const phone = element.querySelector<HTMLAnchorElement>('a[href="tel:+54903416658247"]'); const phone = element.querySelector<HTMLAnchorElement>('a[href="tel:+54903416658247"]');
expect(element.textContent).toContain('CONTACTO'); expect(element.textContent).toContain('CONTACTO');
expect(whatsapp?.textContent).toContain('Chatea con nosotros'); expect(whatsapp?.textContent).toContain('WhatsApp');
expect(
element.querySelector('a[href="https://www.instagram.com/comercio"]')?.textContent,
).toContain('Instagram');
expect(phone?.textContent).toContain('+54 9 (0341) 6658247'); expect(phone?.textContent).toContain('+54 9 (0341) 6658247');
expect(element.textContent).toContain('Rosario'); expect(element.textContent).toContain('Rosario');
expect(element.textContent).toContain('Gigante de Arroyito'); expect(element.textContent).toContain('Gigante de Arroyito');
@@ -123,9 +133,7 @@ describe('ContactPage', () => {
expect(element.querySelector('.contact-page__addresses')?.textContent).toContain( expect(element.querySelector('.contact-page__addresses')?.textContent).toContain(
'Av. Génova 640, Rosario', 'Av. Génova 640, Rosario',
); );
expect(element.textContent).toContain( expect(element.textContent).toContain('Rioja 1150, piso 12, dpto. 3, Rosario');
'Rioja 1150, piso 12, dpto. 3, Rosario',
);
expect(element.textContent).toContain('Santa Fe'); expect(element.textContent).toContain('Santa Fe');
expect(element.textContent).toContain('San Martín 2450'); expect(element.textContent).toContain('San Martín 2450');
}); });
@@ -161,11 +169,35 @@ describe('ContactPage', () => {
}); });
it('shows every canonical address and maps only addresses with coordinates', () => { it('shows every canonical address and maps only addresses with coordinates', () => {
tenantState.set({ ...tenant, addresses: [ tenantState.set({
{ id: 10, label: 'Principal', address_text: 'Nueva sede 123', latitude: 0, longitude: 0, is_main: true }, ...tenant,
{ id: 11, label: 'Sucursal sin mapa', address_text: 'Otra calle 456', latitude: null, longitude: null, is_main: false }, addresses: [
{ id: 12, label: 'Otra ciudad', address_text: 'Tercera 789', latitude: -31, longitude: -60, is_main: false }, {
] }); id: 10,
label: 'Principal',
address_text: 'Nueva sede 123',
latitude: 0,
longitude: 0,
is_main: true,
},
{
id: 11,
label: 'Sucursal sin mapa',
address_text: 'Otra calle 456',
latitude: null,
longitude: null,
is_main: false,
},
{
id: 12,
label: 'Otra ciudad',
address_text: 'Tercera 789',
latitude: -31,
longitude: -60,
is_main: false,
},
],
});
fixture.detectChanges(); fixture.detectChanges();
const element = fixture.nativeElement as HTMLElement; const element = fixture.nativeElement as HTMLElement;
expect(element.textContent).toContain('Nueva sede 123'); expect(element.textContent).toContain('Nueva sede 123');
@@ -188,4 +220,75 @@ describe('ContactPage', () => {
fixture.detectChanges(); fixture.detectChanges();
expect(fixture.nativeElement.querySelectorAll('.contact-page__location')).toHaveLength(0); expect(fixture.nativeElement.querySelectorAll('.contact-page__location')).toHaveLength(0);
}); });
it('updates contact when the resolved social media changes to the active event channels', () => {
tenantState.set({
...tenant,
social_media: [
{
code: 'whatsapp',
name: 'WhatsApp del evento',
icon: 'fa-brands fa-whatsapp',
url: 'https://wa.me/541111111111',
},
],
});
fixture.detectChanges();
const element = fixture.nativeElement as HTMLElement;
expect(element.querySelector('a[href="https://wa.me/541111111111"]')?.textContent).toContain(
'WhatsApp del evento',
);
expect(element.querySelector('a[href="https://wa.me/543412602222"]')).toBeNull();
expect(element.textContent).not.toContain('Instagram');
});
it('shows the main address first without reordering tenant data', () => {
const addresses = [...tenant.addresses!].reverse();
tenantState.set({ ...tenant, addresses });
fixture.detectChanges();
expect(
fixture.nativeElement.querySelector('.contact-page__location-title')?.textContent,
).toContain('Gigante de Arroyito');
expect(addresses[0].id).toBe(3);
});
it('ignores legacy static content and shows the empty state when no canonical data exists', () => {
tenantState.set({
...tenant,
addresses: [],
phone_numbers: [],
social_media: [],
menues: [
{
...tenant.menues![0].submenues[0],
static_content: {
whatsapp: {
whatsapp_url: 'https://wa.me/549999999999',
whatsapp_label: 'Legacy WhatsApp',
},
},
},
],
});
fixture.detectChanges();
expect(fixture.nativeElement.textContent).toContain('No hay datos de contacto disponibles.');
expect(fixture.nativeElement.textContent).not.toContain('Legacy WhatsApp');
expect(fixture.nativeElement.querySelector('a')).toBeNull();
expect(fixture.nativeElement.querySelector('app-location-map')).toBeNull();
});
it('shows social-only contact and ignores blank social URLs', () => {
tenantState.set({
...tenant,
addresses: [],
phone_numbers: [],
social_media: [tenant.social_media![1], { ...tenant.social_media![0], url: ' ' }],
});
fixture.detectChanges();
expect(fixture.nativeElement.textContent).toContain('Instagram');
expect(fixture.nativeElement.textContent).not.toContain(
'No hay datos de contacto disponibles.',
);
expect(fixture.nativeElement.textContent).not.toContain('WhatsApp');
});
}); });

View File

@@ -1,8 +1,8 @@
import { Component, computed, inject } from '@angular/core'; import { Component, computed, inject } from '@angular/core';
import { findMenu } from '../../../../../../core/services/menu.utils';
import { TenantService } from '../../../../../../core/services/tenant.service'; import { TenantService } from '../../../../../../core/services/tenant.service';
import { TenantPhoneNumber } from '../../../../../../core/services/phone-number.interface'; import { TenantPhoneNumber } from '../../../../../../core/services/phone-number.interface';
import { SocialMedia } from '../../../../../../core/services/tenant.interface';
import { MenuContentSectionComponent } from '../../../../components/menu-content-section/menu-content-section.component'; import { MenuContentSectionComponent } from '../../../../components/menu-content-section/menu-content-section.component';
import { import {
LocationMapComponent, LocationMapComponent,
@@ -23,8 +23,8 @@ type ContactAddress = {
}; };
type ContactContent = { type ContactContent = {
whatsappUrl: string | null; whatsapp: SocialMedia[];
whatsappLabel: string | null; socialMedia: SocialMedia[];
phones: TenantPhoneNumber[]; phones: TenantPhoneNumber[];
locations: ContactLocation[]; locations: ContactLocation[];
mapLocations: MapLocation[]; mapLocations: MapLocation[];
@@ -43,53 +43,52 @@ export class ContactPage {
() => this.tenantService.tenant()?.primary_color ?? '#6376f3', () => this.tenantService.tenant()?.primary_color ?? '#6376f3',
); );
protected readonly contact = computed(() => protected readonly contact = computed(() => this.buildContactContent());
this.parseContactContent(
findMenu(this.tenantService.tenant()?.menues ?? [], 'help.contact')?.static_content,
),
);
private parseContactContent(value: unknown): ContactContent { private buildContactContent(): ContactContent {
const content = this.isRecord(value) ? value : {}; const tenant = this.tenantService.tenant();
const whatsapp = this.isRecord(content['whatsapp']) ? content['whatsapp'] : null; const socialMedia = (tenant?.social_media ?? []).filter((social) => social.url.trim());
const addresses = this.tenantService.tenant()?.addresses ?? []; const addresses = [...(tenant?.addresses ?? [])].sort(
(a, b) => Number(b.is_main) - Number(a.is_main),
);
const locations: ContactLocation[] = addresses.map((address) => ({ const locations: ContactLocation[] = addresses.map((address) => ({
id: String(address.id),
label: address.label,
addresses: [
{
id: String(address.id), id: String(address.id),
label: address.label, label: address.label,
addresses: [{ address: address.address_text,
id: String(address.id), coordinates:
label: address.label, address.latitude !== null && address.longitude !== null
address: address.address_text, ? [address.latitude, address.longitude]
coordinates: address.latitude !== null && address.longitude !== null : null,
? [address.latitude, address.longitude] : null, },
}], ],
})); }));
return { return {
whatsappUrl: whatsapp ? this.stringValue(whatsapp['whatsapp_url']) : null, whatsapp: socialMedia.filter((social) => social.code === 'whatsapp'),
whatsappLabel: whatsapp ? this.stringValue(whatsapp['whatsapp_label']) : null, socialMedia: socialMedia.filter((social) => social.code !== 'whatsapp'),
phones: [...(this.tenantService.tenant()?.phone_numbers ?? [])] phones: [...(tenant?.phone_numbers ?? [])].sort(
.sort((a, b) => Number(b.is_main) - Number(a.is_main)), (a, b) => Number(b.is_main) - Number(a.is_main),
),
locations, locations,
mapLocations: locations.flatMap((location) => mapLocations: locations.flatMap((location) =>
location.addresses.flatMap( location.addresses.flatMap((address): MapLocation[] =>
(address): MapLocation[] => address.coordinates === null ? [] : [{ address.coordinates === null
id: address.id, ? []
label: address.label, : [
latitude: address.coordinates[0], {
longitude: address.coordinates[1], id: address.id,
address: address.address, label: address.label,
}], latitude: address.coordinates[0],
longitude: address.coordinates[1],
address: address.address,
},
],
), ),
), ),
}; };
} }
private stringValue(value: unknown): string | null {
return typeof value === 'string' && value.trim() ? value.trim() : null;
}
private isRecord(value: unknown): value is Record<string, unknown> {
return typeof value === 'object' && value !== null && !Array.isArray(value);
}
} }