internxt / drive-desktop

https://internxt.com/drive
GNU Affero General Public License v3.0
168 stars 21 forks source link

[PB-1393]: Feat/add modal to force sync. #498

Closed ArceDanielShok closed 5 months ago

ArceDanielShok commented 6 months ago

This pull request introduces the ModalLogout component, designed to alert users about ongoing processes or unsynchronized files before they log out. The modal includes a button that allows users to force synchronization if needed.

Key Features:

ModalLogout Component: Displays a warning to the user when there are active processes or files that have not yet been synchronized. It offers an option to either proceed with the logout or cancel it to resolve the pending sync issues. Force Sync Button: Allows users to initiate a forced synchronization directly from the modal, helping to ensure that no important data is lost before logging out. Enhancements:

Renderer-Main Communication: Implemented new Electron bindings to facilitate communication between the renderer process and the main process. This allows the modal to fetch and update the status of unsynchronized files effectively. Justification: The addition of this component enhances user experience by providing clear communication regarding the state of their data sync. This ensures that users are fully informed about the state of their work before making critical decisions like logging out.

miguelsw commented 6 months ago

@ArceDanielShok this have 16 issues on analysis, fix it.

miguelsw commented 5 months ago

@ArceDanielShok fix smells

sonarcloud[bot] commented 5 months ago

Quality Gate Failed Quality Gate failed

Failed conditions
C Reliability Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint