fix: 47 errores de lint, y uno de ellos era un bug de verdad
Salieron al pasar el lint por todo el proyecto. De 47 a 0 (quedan 15 avisos: <img> vs <Image /> y el <link> del tema, los dos deliberados). - Footer y Video (42 de los 47, que en realidad eran 7 enlaces: el plugin repite cada uno 6 veces): usaban `<a href="/terms-and-conditions">` SIN el idioma delante. No estaba roto de milagro: el middleware lo salvaba mirando la cookie NEXT_LOCALE. Pero cada clic se comía un 307 y recargaba la página entera en vez de navegar en cliente, y el idioma lo decidía la cookie en vez de la URL en la que estás. Ahora van con el `Link` de i18n, que ya existía. - Turnstile: escribía una ref (`cb.current = onVerify`) DURANTE el render. Es el patrón de "callback fresca", pero no está permitido: React puede descartar ese render y dejarla mal. Pasa a un efecto. - ActivateClient y ConfirmClient: ponían el estado de error con setState dentro del efecto cuando NO hay hash. Eso se sabe ya al renderizar (es un prop), así que se deriva del estado inicial y se ahorra un render. - BattlepayList: `window.location.href = …` -> `.assign()`, igual que en la tienda. - CookieConsent: aquí el efecto es correcto y la regla no aplica, así que se silencia explicando por qué: el consentimiento vive en una cookie del navegador, en el servidor no existe, y leerlo al renderizar rompería la hidratación (le saldría el banner a quien ya había decidido). Verificado: desde /es/ el pie enlaza a /es/terms-and-conditions y desde /en/ a /en/terms-and-conditions; portada, cookies, tienda y mi-cuenta siguen dando 200. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -1,9 +1,14 @@
|
||||
import { getTranslations } from 'next-intl/server'
|
||||
import { getRealmName } from '@/lib/realm'
|
||||
import { Link } from '@/i18n/navigation'
|
||||
|
||||
/**
|
||||
* Pie de página — réplica del partial Django `partials/footer.html`
|
||||
* (enlaces legales + copyright + crédito de diseño) con las clases del tema real.
|
||||
*
|
||||
* Los enlaces van con el `Link` de i18n, no con <a href="/x">: sin el idioma
|
||||
* delante, cada clic se comía un 307 del middleware y recargaba la página
|
||||
* entera, y el idioma lo decidía la cookie en vez de la URL en la que estás.
|
||||
*/
|
||||
export async function Footer() {
|
||||
const t = await getTranslations('UI')
|
||||
@@ -13,27 +18,27 @@ export async function Footer() {
|
||||
return (
|
||||
<footer>
|
||||
<p>
|
||||
<a href="/terms-and-conditions">{t('footer.terms')}</a>
|
||||
<Link href="/terms-and-conditions">{t('footer.terms')}</Link>
|
||||
<span className="third-brown">
|
||||
<i className="fas fa-grip-lines-vertical"></i> {' '}
|
||||
</span>
|
||||
<a href="/privacy-policy">{t('footer.privacy')}</a>
|
||||
<Link href="/privacy-policy">{t('footer.privacy')}</Link>
|
||||
<span className="third-brown">
|
||||
<i className="fas fa-grip-lines-vertical"></i> {' '}
|
||||
</span>
|
||||
<a href="/refund-policy">{t('footer.refund')}</a>
|
||||
<Link href="/refund-policy">{t('footer.refund')}</Link>
|
||||
<span className="third-brown">
|
||||
<i className="fas fa-grip-lines-vertical"></i> {' '}
|
||||
</span>
|
||||
<a href="/cookies">{t('footer.cookies')}</a>
|
||||
<Link href="/cookies">{t('footer.cookies')}</Link>
|
||||
<span className="third-brown">
|
||||
<i className="fas fa-grip-lines-vertical"></i> {' '}
|
||||
</span>
|
||||
<a href="/legal-notice">{t('footer.legal')}</a>
|
||||
<Link href="/legal-notice">{t('footer.legal')}</Link>
|
||||
<span className="third-brown">
|
||||
<i className="fas fa-grip-lines-vertical"></i> {' '}
|
||||
</span>
|
||||
<a href="/contact-us">{t('footer.contact')}</a>
|
||||
<Link href="/contact-us">{t('footer.contact')}</Link>
|
||||
</p>
|
||||
<br />
|
||||
<p className="third-brown">
|
||||
|
||||
Reference in New Issue
Block a user