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





Want the latest updates on software, tech news, and AI?
Get latest updates about software, tech news, and AI from SourceForge directly in your inbox once a month.