Patch for radically simplify the persistence API

[awi...@gmail.com]

+1 to all of these changes.

http://codereview.appspot.com/28142

[bobe...@google.com]

+1

http://codereview.appspot.com/28142

[Louis Ryan]

+1

On Tue, Mar 31, 2009 at 11:25 AM, <bobe...@google.com> wrote:

+1

http://codereview.appspot.com/28142

[Scott Seely]

Lines 388-391 of the new are redundant with the OpenSocial API, section 5.3.12.2.14 newUpdatePersonAppDataRequest. Repetition means we need to maintain the values globally when we update the specs. This is also mentioned in the osapi.appdata section of the patch.

 

Suggestion: Remove the edits about 10 KB globally. Add a sentence to 3. Compliance, stating that the container SHOULD provide 10 KB for AppData.

 

REST-API: XSD is missing a modification to support AppData as an additional element in Person.

[Arne Roomann-Kurrik]

Updated http://codereview.appspot.com/28142 with Scott's feedback.

~Arne

> On Tue, Mar 31, 2009 at 11:25 AM, <bobev...@google.com> wrote:
>
> +1
>
> http://codereview.appspot.com/28142

[Scott Seely]

Looks fine.

We haven't seen a lot of activity on this clarification. Let's plan on committing this at EOD if we see no objections.

[Arne Roomann-Kurrik]

Committed: http://code.google.com/p/opensocial-resources/source/detail?r=1084

~Arne

On Apr 3, 9:18 am, Scott Seely <sSe...@myspace-inc.com> wrote:
> Looks fine.
>
> We haven't seen a lot of activity on this clarification. Let's plan on committing this at EOD if we see no objections.
>
> > -----Original Message-----
> > From: opensocial-an...@googlegroups.com [mailto:opensocial-and-
> > gadget...@googlegroups.com] On Behalf Of Arne Roomann-Kurrik
> > Sent: Thursday, April 02, 2009 3:52 PM
> > To: OpenSocial - OpenSocial and Gadgets Specification Discussion
> > Subject: [opensocial-and-gadgets-spec] Re: Patch for radically simplify the
> > persistence API
>

> > Updatedhttp://codereview.appspot.com/28142with Scott's feedback.