-
Notifications
You must be signed in to change notification settings - Fork 141
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fixes #2120 (plus changes to status info and tray icon active duration) #2142
base: master
Are you sure you want to change the base?
Conversation
Hi, thanks for the PR! Had a quick look and was wondering if it wouldn't be better to use Line 34 in 64586df
Old discussion here: https://www.mail-archive.com/[email protected]/msg16052.html |
@m3nu, Silly me, I should have searched for QThread in all files - was thinking about it but didn't want add an import just for this. Yes of course, I'll try and get my head around utils.py as time permits. |
@m3nu, please hold. I have a working solution that wraps VortaApp.create_backup_action with an Will commit after I've tidied things up next week - need a break and a hair transplant. |
Apologies, tried to make a minor change to 84fbf14 with git commit --amend --no-edit and made a complete hash of it. |
src/vorta/utils.py
Outdated
|
||
def run(self): | ||
self.decorator_instance(self.wrapped_instance, *self.args, **self.kwargs) | ||
<<<<<<< HEAD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Unresolved conflict.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, will action.
Apologies for rogue conflict markers. |
Description
Add pre/post backup signals and emit them in BorgCreateJob.pre_post_backup_cmd.
Use them to toggle start/cancel buttons, tray icon active/inactive and provide enhanced status info. Also run VortaApp.create_backup_action/VortaScheduler.create_backup in a new thread to prevent GUI freezing whilst running pre/post backup commands.
Related Issue
#2120
Motivation and Context
I experienced this issue when running pre/post backup commands to mount/unmount a gdrive.
It solves the problem of the backup finished signal occurring before long running post backup commands have completed, preventing the start/cancel buttons being toggled back to their resting state.
How Has This Been Tested?
By running manual and scheduled backups.
Types of changes
Checklist:
I provide my contribution under the terms of the license of this repository and I affirm the Developer Certificate of Origin.