Skip to content

feat(nfc): C11 NFC platform config, validation, useNfc hook, docs - #38

Merged
JoelVR17 merged 4 commits into
Ding-Payments:developfrom
Rescate-Lacteo:feat/nfc-c11
Jul 20, 2026
Merged

feat(nfc): C11 NFC platform config, validation, useNfc hook, docs#38
JoelVR17 merged 4 commits into
Ding-Payments:developfrom
Rescate-Lacteo:feat/nfc-c11

Conversation

@YAustinXYZ

@YAustinXYZ YAustinXYZ commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Resumen

Consolida soporte y endurecimiento NFC (C11): detección de plataforma, validación de expiración y replay, guardrails de seguridad en el payload, hook useNfc, mapeo de errores en español y documentación operativa.

closes #33

Qué incluye

  • Banner de disponibilidad NFC y deep-link a Ajustes.
  • useNfc() hook (startReading, startWriting, cancel, status).
  • Validación de payment-request (expiry, clock skew 30s, 5-minute dedupe TTL).
  • Guardas de esquema que rechazan campos tipo secret/private/seed/token (test incluido).
  • Mensajes de error centralizados en español (NfcErrorCode).
  • Documentación: docs/nfc-flow.md y docs/nfc-device-checklist.md.
  • Wiring en pantallas Send/Receive y eventos de analytics añadidos.

Acceptance checklist

  • Banner aparece y abre ajustes si NFC deshabilitado.
  • Expiry y dedupe funcionan (unit + manual).
  • Tests de esquema fallan si se introducen campos sensibles.
  • useNfc() expone la API estable y limpia sesiones al unmount.
  • No se envían payloads/secretos en eventos de analytics.

Notas

  • NfcReader actualmente es un stub/simulador; reemplazar por react-native-nfc-manager al integrar en dispositivo.
  • Rebuild nativo necesario tras cambios en app.config.ts y plugins.

Para probar localmente:

# build/typecheck/tests
npm run build
npm run test
# dev build for NFC
npx expo prebuild --clean
npx expo run:ios
npx expo run:android

Closes: C11

@JoelVR17
JoelVR17 merged commit e1f59bd into Ding-Payments:develop Jul 20, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[C11] NFC platform config, security and documentation

2 participants