Conversation
update CVE Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
Updated Node.js and npm engine requirements and upgraded nodemailer types. Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
* Add avatar upload with size limit and streaming Implemented streaming upload for avatar with size limit. Signed-off-by: Damien <mouette03@yahoo.fr> * Update settings.ts Signed-off-by: Damien <mouette03@yahoo.fr> * Improve error handling for avatar upload Handle specific error cases for avatar upload. Signed-off-by: Damien <mouette03@yahoo.fr> * Improve error handling for logo upload Enhance error handling for logo upload by checking response status and code. Signed-off-by: Damien <mouette03@yahoo.fr> * Improve file upload size validation and error handling Refactor file upload handling to check size limits during write process and handle errors more gracefully. Signed-off-by: Damien <mouette03@yahoo.fr> --------- Signed-off-by: Damien <mouette03@yahoo.fr>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com> Signed-off-by: Damien <mouette03@yahoo.fr>
Signed-off-by: Damien <mouette03@yahoo.fr>
Docstrings generation was requested by @Mouette03. * #4 (comment) The following files were modified: * `backend/src/routes/auth.ts` * `backend/src/routes/settings.ts` * `backend/src/routes/uploadRequests.ts` * `frontend/src/pages/ProfilePage.tsx` * `frontend/src/pages/SettingsPage.tsx` Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (5)
📝 WalkthroughCheminementAjouts de commentaires JSDoc documentant les fonctions exportées pour les routes d'authentification, paramètres et demandes de téléchargement, ainsi que pour les composants de pages de profil et paramètres. Suppression d'un caractère BOM superflu. Aucune modification de logique, flux de contrôle ou comportement fonctionnel. Modifications
Estimation de l'effort de révision du code🎯 1 (Trivial) | ⏱️ ~3 minutes Poème
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Signed-off-by: Damien <mouette03@yahoo.fr>
This pull request introduces several improvements across the backend and frontend, focusing on file upload robustness, dependency updates, and code documentation. The main highlights include stricter server-side file size enforcement for avatars, logos, and upload requests, user-facing error handling for oversized uploads, dependency upgrades for better compatibility and security, and enhanced inline documentation for key routes and pages.
Backend Improvements:
File Upload Robustness
backend/src/routes/auth.ts).backend/src/routes/settings.ts).backend/src/routes/uploadRequests.ts).Dependency and Environment Updates
backend/package.json).package.jsonfor environment consistency (backend/package.json).Documentation
auth.ts,settings.ts,uploadRequests.ts) to clarify endpoint purposes and side effects. [1] [2] [3]Docker Build Optimization
.github/workflows/docker.yml). [1] [2]Frontend Improvements:
User Feedback for File Uploads
frontend/src/pages/ProfilePage.tsx,frontend/src/pages/SettingsPage.tsx). [1] [2]Documentation
ProfilePage.tsxandSettingsPage.tsxcomponents to describe their purpose and behavior. [1] [2]Summary by CodeRabbit
Documentation
Chores