No Problem Ive got a example file its not submitted yet as it had my keys in and I was using it , I'll clean that up and put that into the repo as well. I'll should finish all the descriptor files this weekend and once that is done all the test can be finished. I just got behind of with some other work and not had much time this last week to work on it but Yes all Tests and Example will be added.
October 31 2014 7:53 AM, "Rod Simpson" <r...@rodsimpson.com> wrote: > Jason, > > I finally had a chance to go though the code today. Looks like you have made > a lot of great > progress. The code is clean and well written and it looks like you have > started adding unit tests. > > One thing that I didn’t see were any examples. In an effort to try to make > our SDKs as easy as > possible to pick up and use, we have tried to add samples to each one. Not > all the SDKs have this, > but the ones that do have become much more popular since it is easier for > people to get started. > For example, the JavaScript SDK has quite a few: > > https://github.com/apache/incubator-usergrid/tree/master/sdks/html5-javascript/examples > > It would be awesome if you could add just a couple of samples so people will > see how to get up and > running. I see that you have added some getting started info into the readme > file, which is great, > but code examples really go a long way towards getting a fellow developer up > and running. > > Apart from that, I think this is really good stuff and will be a great > contribution to the Apache > Usergrid project! > > Thanks! > > Rod > > -- > Rod Simpson > @rockerston > rodsimpson.com > > On October 28, 2014 at 9:56:49 PM, jas...@apps4u.com.au > (jas...@apps4u.com.au) wrote: > > Hi Im happy to change or remove the copy right it was a set template in my > IDE so I can either > remove copyright or just have copy right to myself and remove all mention of > baas-platform.com .. I > won baas-platform.com so I can remove copyright if needed .. I just didn't > want to change iDE > templates as they took a while to setup. > > October 29 2014 12:31 AM, "Dave" wrote: > > Thanks Jason! > > I don't know PHP so I'm not the best person to review this pull request, but > I did take a look. > The project looks well-thought out and I did not see any red flags like > binary files or missing > headers. > > Here's the PR: https://github.com/apache/incubator-usergrid/pull/88 > (https://github.com/apache/incubator-usergrid/pull/88) > > Here are my questions for the team: > > 1) We already have a PHP SDK, how is this one different or better than what > we have now? > > 2) This is a big contribution, assuming it looks good to others: should we > simply pull it in to > Usergrid master -- or do we need to do a code grant? > > 3) The files have an ASL license header and also a copyright by > baas-platform.com > (http://baas-platform.com). My understanding is that we do not require > copyright assignment, so > that's fine, no? > > Thoughts? > > Any PHP experts out there want to weigh in on this contribution? Who wrote > our existing PHP SDK? > > Thanks, > Dave > > On Sun, Oct 26, 2014 at 1:55 AM, apps4u wrote: > > Github user apps4u commented on the pull request: > > https://github.com/apache/incubator-usergrid/pull/88#issuecomment-60507519 > (https://github.com/apache/incubator-usergrid/pull/88#issuecomment-60507519) > > Added some more api operation descriptors to version 1.0.1 of this SDK . It > now has cleaner api and > ResourceIteration as well.
