considering valid_thru on points report inside customer page

[vitorrgg]

No description provided.

[netlify]

✅ Deploy Preview for ecomplus-admin-dev ready!

Name	Link
🔨 Latest commit	51fad8d
🔍 Latest deploy log	https://app.netlify.com/projects/ecomplus-admin-dev/deploys/6988e41bbd81bb0009d97247
😎 Deploy Preview	https://deploy-preview-906--ecomplus-admin-dev.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[netlify]

✅ Deploy Preview for ecomplus-admin ready!

Name	Link
🔨 Latest commit	51fad8d
🔍 Latest deploy log	https://app.netlify.com/projects/ecomplus-admin/deploys/6988e41be984ba0008805421
😎 Deploy Preview	https://deploy-preview-906--ecomplus-admin.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[leomp12]

@claude deixe um review sucinto aqui

[claude]

Claude finished @leomp12's task in 59s —— View job

---

Review do PR

• Ler as alterações no código
• Analisar a lógica de valid_thru
• Fornecer feedback

---

Análise

As alterações implementam corretamente a verificação de expiração de pontos com base no campo valid_thru. O código:

Pontos positivos:

1. ✅ Adiciona indicação visual de pontos expirados (opacidade reduzida + label "expirado")
2. ✅ Exclui pontos expirados do total de activePoints no cálculo do rodapé
3. ✅ Mantém pontos expirados no total de earnedPoints (correto do ponto de vista contábil)
4. ✅ Ordenação por valid_thru garante que pontos mais antigos apareçam primeiro

Observações:

• A lógica está consistente: verifica expiração em dois lugares (renderização da linha e cálculo do total)
• O cálculo new Date(entry.valid_thru) < now é adequado para verificar expiração
• A UI fica clara para o usuário sobre quais pontos estão expirados

Sugestão menor (opcional):
No arquivo src/controllers/resources/form/customers.js:360, a verificação de expiração é duplicada (linhas 360 e 381). Poderia ser otimizado calculando isExpired uma vez para cada entry antes do .forEach(), mas como os dados são pequenos, o impacto de performance é negligenciável.

Conclusão

✅ Código aprovado. A implementação está correta e resolve o problema de considerar a data de validade dos pontos no relatório.

---