Menu

#42 refactor NJClient settings and remote user/channel creation

closed
nobody
2014-07-03
2014-06-27
Anonymous
No

Originally created by: bill-auger
Originally owned by: bill-auger

feature of mini-epic feature [#34]

the LinJamConfig::valueTreePropertyChanged() event handler could be simplified greatly by applying law of demeter and relegating the NJClient settings to LinJam class as is done for local channels

  • refactor LinJam::ConfigureRemoteChannel() to load from storage like LinJam::ConfigureLocalChannel()
  • refactor LinJam::ConfigureRemoteUser() to load from storage the same
  • refactor LinJamConfig::valueTreePropertyChanged() accordingly
  • create LinJam::AddRemoteChannel() like LinJam::AddLocalChannel() for LinJam::HandleUserInfoChanged() and for creating remote master mixer (issue [#34]) - this should also ensure that the canonical LinJamConfig::encodeChannelId() happens in a single place (LinJamConfig::createChannel())

Related

Tickets: #34

Discussion

  • Anonymous

    Anonymous - 2014-06-30

    Ticket changed by: bill-auger

    • status: open --> closed
     
  • Anonymous

    Anonymous - 2014-06-30

    Originally posted by: bill-auger

    done

    [3aff98]
    [e7a333]

     

Anonymous
Anonymous

Add attachments
Cancel