Revisar la manera de no publicar LivenessMovement
🎯 Summary
Al hacer MOBBSCAN-5961 en iOS, se ha añadido la propiedad livenessMovement (del nuevo tipo MobbScanLivenessMovement) a la API pública de MobbScan. Revisar si esto es realmente necesario o si hay alguna forma mejor de hacerlo.
- Hice esto principalmente para poder mostrar el String localizado para cada movimiento desde MobbScan_MobbFaceUX.
- He creado esta tarea porque Juan Andrés Borrallo Domínguez comentó en la MR por qué era necesario esto, y me ha hecho dudar si hay alguna forma mejor de hacerlo.
🤔 Type
-
✨ New feature -
🐛 Bug fix -
👷 Technical debt -
☢ ️ Breaking change (it breaks backward compatibility) -
📚 Requires a change in the docs
⚙ ️ Configuration
Workspace branch: [feature/MOBBSCAN-6037]
Related MRs:
-
🔗 [https://gitlab.mobbeel.com/mobbscan/MobbScanSDK-iOS/-/merge_requests/157] -
🔗 [https://gitlab.mobbeel.com/mobbscan/mobbscan_mobbfaceux-ios/-/merge_requests/9]
⚗ ️ Solution
Marcada la propiedad livenessMovement de MobbScanAPI como de solo lectura (de forma que puede ser usada por MobbScan_MobbFaceUX para mostrar el texto del movimiento aleatorio actual), pero reimplementada internamente dentro de MobbScan_UnatttendedPlugin como de lectura-escritura (de forma que, al recibir su valor del backend en la llamada al startVideo, se puede asignar su valor correctamente).
-
🔗 JIRA: MOBBSCAN-6037
🔬 Tests
- Comprobar que el proyecto compila (no hay cambios a nivel funcional).
- Comprobar que desde una app que use MobbScanSDK no se puede modificar el
MobbScanLivenessMovement(por ejemplo con el comandoMobbScanAPI.getInstance().livenessMovement = MobbScanLivenessMovement_YES_WITH_YOUR_HEAD).
🧐 Checklist:
- The code follow the project guidelines.
- The changes have been tested locally.
- The code has internal docs.
- Docs are updated (README, public docs, etc.)
- Some tests have been added to check that the changes work as expected (unit, integration or e2e tests ...)
- Unit tests are running without errors.
- Other dependant changes have already been merged.
- Changelog updated (if it applies)