← Back to team overview

cf-charmers team mailing list archive

Re: [Merge] lp:~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context into lp:~cf-charmers/charm-helpers/cloud-foundry

 

The idea here is fine. I'd suggest that this is a different context type as it implies a different relationship to the template is manages however. Any consumer/template needs to be prepared to iterate a list. 

Maybe a subclass called RelationListContext which sets multiple units to True and the base class doesn't expose a way to change this directly, set to False in the class. That way we share the implmentation but it is explicit to the template what they will be consuming.


-- 
https://code.launchpad.net/~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context/+merge/220508
Your team Cloud Foundry Charmers is requested to review the proposed merge of lp:~lomov-as/charm-helpers/cloud-foundry-multiple-units-in-relation-context into lp:~cf-charmers/charm-helpers/cloud-foundry.


Follow ups

References