search

OpenChemistry / avogadroapp

Avogadro is an advanced molecular editor designed for cross-platform use in computational chemistry, molecular modeling, bioinformatics, materials science, and related areas.
https://two.avogadro.cc/
BSD 3-Clause "New" or "Revised" License
179 stars 69 forks source link

AUTOSAVE FEATURE (#182) #486

Closed vinayakjeet closed 3 months ago

vinayakjeet commented 3 months ago

WIP (#182 ) Title: Implement Autosave Feature for Avogadro

Description: This pull request introduces a new autosave feature to the Avogadro application. The feature aims to enhance user experience by automatically saving the current molecule document at a specified interval, reducing the risk of data loss in case of unexpected application closure. The implementation includes setting up an autosave interval with QTimer, determining the autosave directory based on the application data location, and saving the document in the CJSON format. This autosave functionality is careful to activate only if there are unsaved changes, ensuring efficient use of resources and avoiding unnecessary writes. The feature also involves updates to the MainWindow class to incorporate autosave setup and execution logic. This addition is expected to provide a safety net for users, securing their work without requiring manual intervention for frequent saves.

Signed-off-by: vinayakjeet vinayakjeetog@gmail.com

vinayakjeet commented 3 months ago

Hello mentors @ghutchis , the pull request is failing due to non-compliance with DCO format despite adhering to the guidelines in the commit message line adjustments. Seeking guidance on resolving this issue efficiently. Thank you

matterhorn103 commented 3 months ago

Easiest thing for me when I had the same issue was just to make a fresh branch, make the same changes to the code, and make a new git commit. Signing off is very easy once you know how, just make sure use the -s flag for every commit e.g. git commit -s -m "Commit message."

As far as I can remember you effectively agree to the DCO by signing off commits like this and you don't need to do anything else.

Then you can open a new pull request from the new branch and the DCO check will be fine.