Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

IOS Registering to topics doesn't work #706

Closed
jakari opened this issue Mar 16, 2016 · 4 comments
Closed

IOS Registering to topics doesn't work #706

jakari opened this issue Mar 16, 2016 · 4 comments

Comments

@jakari
Copy link
Contributor

jakari commented Mar 16, 2016

Expected Behaviour

As per the API.md mentions, you should be able to register to topics in IOS, by adding topics into init:

var push = PushNotification.init({
    ios: {
        senderID: "12345679",
        gcmSandbox: true,
        alert: "true",
        badge: true,
        sound: 'false',
        topics: ['foo', 'bar']
    }
});

Actual Behaviour

When I try to register to topics in IOS like in the example, nothing happens, the system doesn't register to given topics in GCM.

I was confused, so I began digging the source code, and it seems there is nothing that interprets the topics array on the IOS side, so the feature seems to be missing altogether.

I digged deeper, and it seems there is a issue #79 with a branch called issue79 that has this functionality but it isn't merged to master.

Is there some reason why this functionality isn't merged to master?

@macdonst
Copy link
Member

@jakari Nope, that seems like a screw up on my part. I'm on vacation right now but will look at it within a week.

@jakari
Copy link
Contributor Author

jakari commented Mar 17, 2016

I actually forked and patched it in already since I needed the functionality, so if you don't mind, I can make a pull request of it and see what you think.

@macdonst
Copy link
Member

macdonst commented Mar 17, 2016 via email

@lock
Copy link

lock bot commented Jun 5, 2018

This thread has been automatically locked.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 5, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

No branches or pull requests

2 participants