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

[PubNub setConfiguration:config]; on background thread doesn't work #26

Closed
mcmurrym opened this issue Aug 7, 2013 · 7 comments
Closed
Assignees

Comments

@mcmurrym
Copy link

mcmurrym commented Aug 7, 2013

If I call [PubNub setConfiguration:config]; on a background thread, then when I call connect, pubnub never connects.

@geremyCohen
Copy link
Contributor

Hi Matt-

Can you send in a sample XCode app to support@pubnub.com that demonstrates
the issue, and we'll help you resolve the issue from there?

On Tue, Aug 6, 2013 at 8:58 PM, Matt notifications@github.com wrote:

If I call [PubNub setConfiguration:config]; on a background thread, then
when I call connect, pubnub never connects.


Reply to this email directly or view it on GitHubhttps://github.com//issues/26
.

@ghost ghost assigned parfeon Aug 7, 2013
@mcmurrym
Copy link
Author

mcmurrym commented Aug 7, 2013

I sent a sample project. but here is an image snippet.
image

@geremyCohen
Copy link
Contributor

@mcmurrym is this a blocker for you, or are you able to continue development while we work on a solution for you? Please let me know priority so we can schedule this for you appropriately.

@mcmurrym
Copy link
Author

mcmurrym commented Aug 8, 2013

I bumped it to the main thread for that procedure which will be just fine.
No rush for me.

Sent from my PiePhone mmmm pie

On Aug 8, 2013, at 5:00 PM, Geremy Cohen notifications@github.com wrote:

@mcmurrym https://github.com/mcmurrym is this a blocker for you, or are
you able to continue development while we work on a solution for you?
Please let me know priority so we can schedule this for you appropriately.


Reply to this email directly or view it on
GitHubhttps://github.com//issues/26#issuecomment-22364670
.

@geremyCohen
Copy link
Contributor

Safe to close this ticket out then? Or leave it open until we discover a solution for you?

@mcmurrym
Copy link
Author

mcmurrym commented Aug 8, 2013

It is my opinion that you should either document that this must be done on
the main thread or else allow it to work on the background thread. Whether
or not there is a workaround it is a bug in my mind. Ill let you decide to
close it or not

Sent from my PiePhone mmmm pie

On Aug 8, 2013, at 5:09 PM, Geremy Cohen notifications@github.com wrote:

Safe to close this ticket out then? Or leave it open until we discover a
solution for you?


Reply to this email directly or view it on
GitHubhttps://github.com//issues/26#issuecomment-22365065
.

@geremyCohen
Copy link
Contributor

Good point @mcmurrym I will document that this must be done on the main thread :) I will close this ticket for now, but will update the docs with this info as well shortly.

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

3 participants