client icon indicating copy to clipboard operation
client copied to clipboard

Add signal to Account for logged in

Open TheOneRing opened this issue 2 years ago • 2 comments

AccountState has the logged in state but this is only available in the ui code. The spaces manager misuse the credentialsFetched signal but thats a bit hacky.

TheOneRing avatar Apr 20 '23 11:04 TheOneRing

I think it would be better to either move the SpacesManager to the AccountState (as it's only used when there is a state), or connect it (directly, or indirectly through a lambda) to the AccountState::stateChanged signal.

erikjv avatar Oct 10 '24 15:10 erikjv

The SpacesManager is used in the cmdline client, so moving its ownership is not an option. Going for #2: connect the slot from the outside when creating an AccountState

erikjv avatar Oct 17 '24 14:10 erikjv