search

nmjukin / google-sites-liberation

Automatically exported from code.google.com/p/google-sites-liberation
0 stars 0 forks source link

Code Review Request #4

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
Review branch: base

Can someone (I'm guess JJ?) take a look at what I have up here so far?
SiteExporter and PageExporter clearly aren't the structure I want them to
be in yet, but ContinuousContentFeed seems good and I also want to know if
the XmlElement and HyperLink classes seem like a reasonable way to do the
html generation (I looked at third party options for that but they all
seemed unnecessarily complicated for what I need).

Original issue reported on code.google.com by yourfriendben@gmail.com on 29 Jun 2009 at 5:45

GoogleCodeExporter commented 8 years ago 
Yeah, I'll take a look at it, review it, and try it out today and tomorrow.  
Thanks!

Original comment by jlu...@gmail.com on 29 Jun 2009 at 6:24

GoogleCodeExporter commented 8 years ago 
I'll take a look too.

Original comment by luke.bla...@gmail.com on 30 Jun 2009 at 4:23

GoogleCodeExporter commented 8 years ago 
I think I've responded to everyone's comments now, so if you could take a look 
at the
latest in the "base" branch that'd be great.

Original comment by yourfriendben@gmail.com on 1 Jul 2009 at 7:39

GoogleCodeExporter commented 8 years ago

Original comment by jlu...@gmail.com on 1 Oct 2009 at 5:57