Code review comment for lp:~salgado/lazr.restful/extension-interfaces

Revision history for this message
Guilherme Salgado (salgado) wrote :

On Wed, 2010-08-04 at 18:09 +0000, Gary Poster wrote:
> >
> > After letting this marinate on my head for a while longer, I thought of
> > creating an IHasComments interface that contributes a 'comments'
> > attribute to IRecipe, as if there was a separate web service which wants
> > to allow its users to comment on recipes
> > (http://paste.ubuntu.com/472821/). How does that sound to you?
>
> I like it.
>
> You *could* show what I was just talking about above, if you wanted to, by setting up a separate webservice that added comments to the original webservice. It ought to be super easy--include the configuration of the original webservice plus your extra bits. If it's not easy then that might even be construed as a problem.
>
> But that's not a requirement for this to land.
>

I've tried that by doing the same thing that the other examples do, but
it didn't work: http://paste.ubuntu.com/473253/

http://paste.ubuntu.com/473254/ has my changes, but I've also pushed
them to my branch. Any idea why it's not working?

--
Guilherme Salgado <https://launchpad.net/~salgado>

« Back to merge proposal