fix(apps): address functions deployment compatibility and resolve callable signature issues#3565
Conversation
There was a problem hiding this comment.
Code Review
This pull request significantly improves the application's architecture and security by moving data access to backend callable functions and locking down Firestore. The changes are well-structured and address important deployment compatibility issues. I've found one high-severity issue in the new updateUser function where handling for indefinite blocks is incorrect, which could lead to storing invalid data. A fix is suggested.
91e5589 to
c816db4
Compare
…lable signature issues
c816db4 to
a14d4bf
Compare
|
This PR was merged into the repository. The changes were merged into the following branches:
|
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Includes refactored callable return structure and CommonJS deployment stabilizing fixes.