search

ActivityWatch / aw-qt

Tray icon that manages ActivityWatch processes, built with Qt.
Mozilla Public License 2.0
27 stars 29 forks source link

Dev/improved exit handling #55

Closed xylix closed 4 years ago

xylix commented 4 years ago

Remove unnecessary exit confirmation (Fixes https://github.com/ActivityWatch/activitywatch/issues/359 )

Actually use the manager to stop all modules

xylix commented 4 years ago

Oh right also had some other problems like that in the rows using the old exit fn. Fixed in bd20f05.

(I hadn't yet run the code locally, shouldn't really push a PR before I've done that whoops)

Will also need to move the signal.signal(signal.SIGINT, lambda: exit(manager)), signal.signal(signal.SIGTERM, lambda: exit(manager)) rows into somewhere with access to self to have them call the actual cleanupping exit and not the system exit function. (Not actually a problem, they only need access to the manager which is passed as an argument to run.