You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@usergrid.apache.org by apps4u <gi...@git.apache.org> on 2014/11/21 08:49:06 UTC
[GitHub] incubator-usergrid pull request: Updated response models, added al...
GitHub user apps4u opened a pull request:
https://github.com/apache/incubator-usergrid/pull/104
Updated response models, added all push notification endpoints. Added new feature called Attributes to access related models as a property on the response model, added typed response models for all default collections.
This includes more polish so all notification endpoints are added and you can send notification to user, users, group, groups, device, devices.
Relationship Attributes so no need to make a 2nd api request to get a default relationship just access the relationship as a property on the response model e.g.: ```$user = Usergrid::users()->findById(['uuid' => '1232']); $device = $user->device; or $groups = user->groups; ```
This make the the api easier to use.
Also added typed collections for all default collections and when asking a response model for its entities property instead of getting the Base PHP Collection class you now get a typed Collection class so when asking a users response model for its entities I get a Users collection class back.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apps4u/incubator-usergrid master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-usergrid/pull/104.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #104
----
commit b06a805f1fd2ff5891fe2d8382a83064c6c2b0e3
Author: Jason Kristian <ja...@apps4u.com.au>
Date: 2014-11-12T06:06:52Z
update composer to stable from dev.
commit 9ffe2b0d4d468b7d7c28f20f50d7e40afbe96c41
Author: Jason Kristian <ja...@apps4u.com.au>
Date: 2014-11-21T07:41:12Z
Create Attributes . So now when I have a User entity I don't need to call the api a 2nd time for the device relationship I can get access the device property on the User model e.g.: $user->device; and the SDK will take care of the 2nd API call on my behalf.
Also added new response models so groups and roles and notifications and notifiers are all typed model response so they no longer return a entity model but a correctly typed models e.g.: Roles returns a roles model.
Also when asking a Response model for its entities property it now returns the correct model of the correct type: e.g. before this change when asking a user model for its entity you would get the base php collection model class now you get the Typed PHP Collection class so asking a User response for its entities property you get back a Users PHP collection class not BaseCollection class.
All notification endpoints are now mapped. You can send to user , users, group, groups, device, devices can all be targeted for a push notiifcation.
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Updated response models, added al...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/incubator-usergrid/pull/104
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Updated response models, added al...
Posted by apps4u <gi...@git.apache.org>.
Github user apps4u commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/104#issuecomment-64293923
This is for the PHP SDK only no other files are changed . I just didn't added this till after I submitted the new PHP SDK. Ive made sure that the same headers are in theses files as the PHP SDK and there is not binary files ... :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] incubator-usergrid pull request: Updated response models, added al...
Posted by rodsimpson <gi...@git.apache.org>.
Github user rodsimpson commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/104#issuecomment-66557569
This has been Acked - sorry it took so long (it was a big PR!).
Rod
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---