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.

Sign up for GitHub

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

Jump to bottom

ARTPushActivationState Crash when call ably.push.activate #628

Closed
Chris59160 opened this issue Aug 19, 2017 · 1 comment
Closed

ARTPushActivationState Crash when call ably.push.activate #628

Chris59160 opened this issue Aug 19, 2017 · 1 comment
Assignees
@tcard

Comments

@Chris59160
Copy link

Hi guys,

I try to use push notification with Ably on iOS.

I have pulled : pod 'Ably', :git => '/~https://github.com/ably/ably-ios.git', :branch => 'push'

I do

    let client:ARTRealtime = self.getAblyRealtime()
    client.push.activate()

on the second line, the app crashes straight away on:

2017-08-19 15:58:22.488 comedy_speed_dating[428:162997] *** Assertion failure in -[ARTPushActivationState transition:], /Users/christophedanguien/Documents/dev/thorge software/ios/comedy_speed_dating/Pods/Ably/Source/ARTPushActivationState.m:38
2017-08-19 15:58:22.491 comedy_speed_dating[428:162997] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: '-[/Users/christophedanguien/Documents/dev/thorge software/ios/comedy_speed_dating/Pods/Ably/Source/ARTPushActivationState.m:38 -[ARTPushActivationState transition:]] should always be overriden.'

Any idea why it's doing that ?

Thank you,

Chris
@tcard tcard self-assigned this Aug 21, 2017
@tcard
Copy link
Contributor

tcard commented Aug 23, 2017

@Chris59160 I've just upgraded the push branch with a fix for this, plus lots of updates from the master branch. Can you confirm this isn't happening anymore?

@tcard tcard closed this as completed Aug 25, 2017
tcard added a commit that referenced this issue Sep 15, 2017
@tcard
ARTPushActivationState: use init instead of new.
eb20143 
With a static method, the constructed class is always the parent
abstract class ARTPushActivationState, instead of the desired child
class.

Fixes #628.
tcard added a commit that referenced this issue Oct 11, 2017
@tcard
ARTPushActivationState: use init instead of new.
0468ff2 
With a static method, the constructed class is always the parent
abstract class ARTPushActivationState, instead of the desired child
class.

Fixes #628.
funkyboy pushed a commit that referenced this issue Jul 17, 2018
@tcard @funkyboy
ARTPushActivationState: use init instead of new.
9d0c615 
With a static method, the constructed class is always the parent
abstract class ARTPushActivationState, instead of the desired child
class.

Fixes #628.
funkyboy pushed a commit that referenced this issue Jul 27, 2018
@tcard @funkyboy
ARTPushActivationState: use init instead of new.
4fb7432 
With a static method, the constructed class is always the parent
abstract class ARTPushActivationState, instead of the desired child
class.

Fixes #628.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tcard tcard

Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tcard @Chris59160