Merge ~sylvain-pineau/checkbox-ng:fix-session-type into checkbox-ng:master
Proposed by
Sylvain Pineau
Status: | Merged |
---|---|
Approved by: | Sylvain Pineau |
Approved revision: | fa3949881890c7fccab10919d189826770f437a2 |
Merged at revision: | cb6088e126c805dde0f4ebb326a7ef826875e9d7 |
Proposed branch: | ~sylvain-pineau/checkbox-ng:fix-session-type |
Merge into: | checkbox-ng:master |
Diff against target: |
32 lines (+9/-5) 1 file modified
plainbox/impl/session/assistant.py (+9/-5) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Maciej Kisielewski | Approve | ||
Checkbox Developers | Pending | ||
Review via email: mp+417976@code.launchpad.net |
Description of the change
checkbox remote is broken (again) since:
https:/
The way the session type is extracted from session titles does not work as launchers are setting
a different value causing the type to fallback to "local".
The patch evaluates the session type based on the session metadata app blob instead.
Tested using the dev ppa version (to verify the behavior of the deb restart strategy) and this patch applied manually (on jammy beta)
To post a comment you must log in.
Neverending story :)
LGTM, +1