Skip to content

feat(store): enforce dynamic rendering and comment out static configu… #106

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Stivenjs merged 1 commit into from
Jun 7, 2025
Merged

feat(store): enforce dynamic rendering and comment out static configu… #106

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 11 additions & 13 deletions app/[store]/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ import { Metadata } from 'next'
import { notFound } from 'next/navigation'
import { storeRenderer } from '@/lib/store-renderer'

// Forzar renderizado dinámico para acceder a variables de entorno en runtime
export const dynamic = 'force-dynamic'

interface StorePageProps {
params: Promise<{
store: string
Expand Down Expand Up @@ -93,7 +96,7 @@ export async function generateMetadata({
: undefined,
}
} catch (error) {
console.error(`Error generating metadata for ${store}${path}:`, error)
console.error(`[Metadata] ERROR generating metadata for ${store}${path}:`, error)

// Metadata por defecto para errores
return {
Expand All @@ -104,17 +107,12 @@ export async function generateMetadata({
}

/**
* Configurar revalidación de páginas para ISR
* Esto permite que las páginas se regeneren automáticamente
* COMENTADO: Estas configuraciones estáticas conflictan con force-dynamic
* No se pueden usar juntas en Next.js 15
*/
export const revalidate = 1800 // 30 minutos

/**
* Configurar generación estática para tiendas populares
* (esto se ejecutaría en build time)
*/
export async function generateStaticParams() {
// TODO: Obtener lista de tiendas activas desde la base de datos
// Por ahora retornamos array vacío para generar páginas bajo demanda
return []
}
// export const revalidate = 1800 // 30 minutos

// export async function generateStaticParams() {
// return []
// }
21 changes: 10 additions & 11 deletions app/[store]/products/[product]/page.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,9 @@ import { Metadata } from 'next'
import { notFound } from 'next/navigation'
import { storeRenderer } from '@/lib/store-renderer'

// Forzar renderizado dinámico para acceder a variables de entorno en runtime
export const dynamic = 'force-dynamic'

interface ProductPageProps {
params: Promise<{
store: string
Expand Down Expand Up @@ -113,16 +116,12 @@ export async function generateMetadata({ params }: ProductPageProps): Promise<Me
}

/**
* Configurar ISR para productos
* Los productos cambian con más frecuencia que las páginas principales
* COMENTADO: Estas configuraciones estáticas conflictan con force-dynamic
* No se pueden usar juntas en Next.js 15
*/
export const revalidate = 900 // 15 minutos

/**
* Generar parámetros estáticos para productos populares
*/
export async function generateStaticParams() {
// TODO: Implementar generación de productos populares
// Por ahora generar bajo demanda
return []
}
// export const revalidate = 900 // 15 minutos

// export async function generateStaticParams() {
// return []
// }