Code review comment for lp://staging/~brendan-donegan/checkbox/story871_session_restore_dbus

Revision history for this message
Daniel Manrique (roadmr) wrote :

yes, of course +1 since all this code has been reviewed and ping-ponged to death.

For the record, there are horrible KLUDGES in this to make things work, the logic behind them seems to be sound but the real, clean solution would be architecturally more involved. However for the purposes of getting the API out there, this is good enough.

Thanks for working on this!

review: Approve

« Back to merge proposal