Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Targeted Push not working #167

Closed
ArifADS opened this issue Oct 24, 2015 · 3 comments
Closed

Targeted Push not working #167

ArifADS opened this issue Oct 24, 2015 · 3 comments

Comments

@ArifADS
Copy link

ArifADS commented Oct 24, 2015

I used the PHP SDK for sending Push Notification last year and worked perfectly, but I updated the SDK and my code is not working this year, so i went back to the old version and seems to work great, I'm doing it like it says in the Docs.

My Code:

$queryIOS = ParseInstallation::query();
$queryIOS->equalTo('start', true);
$queryIOS->containedIn("channels", [$t1,$t2,"dev"]);

ParsePush::send(array(
  "where" => $queryIOS,
  "data" => array(
       "alert" => $startString,
       "category" => "PbP2",
       "gameID" => $match{"id"}
  )
));

any problem seen?

stoiev referenced this issue Oct 26, 2015
Parse pusher supports pushing to all installations at once using "where" : "{}", however with php sdk it's not possible, if you don't pass a where query it will throw "'Missing the push channels.", if an empty ParseQuery is used, "undefined index where" is thrown. This patch aim to pass a "{}" if an empty parse query object is passed.
@stoiev
Copy link
Contributor

stoiev commented Oct 26, 2015

It seens a typo on 8e7d135. I faced it too.

@gfosco
Copy link
Contributor

gfosco commented Oct 26, 2015

Good spot @stoiev ! Can you send in a PR to address it?

@sahanh
Copy link
Contributor

sahanh commented Oct 27, 2015

What a mistake, please accept my PR.

gfosco added a commit that referenced this issue Oct 27, 2015
variable naming fix. regarding #165 #167
@gfosco gfosco closed this as completed Oct 27, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants