Code review comment for lp://qastaging/~lihuiguo/landscape-charm/registration-relation

Revision history for this message
Simon Poirier (simpoir) wrote :

Never mind about the dashboard. I was missing the homer resource in my test bundle.
Looking back at it, I think the proposed code would be more consistent with the existing code if we were using a data provider like for other relations code.

Do you think the following refactor to your branch would make sense? https://pastebin.canonical.com/p/vffVXMCb3y/

« Back to merge proposal