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

Removed push token handling to respect RSH3a3a #534

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

maratal
Copy link
Collaborator

@maratal maratal commented Jul 25, 2024

Removed unnecessary push token handling. Token is requested by ably-cocoa and received by flutter, which sends it to ably via the ARTRealtime object, through which the token was requested.

Closes #533

@github-actions github-actions bot temporarily deployed to staging/pull/534/features July 25, 2024 15:52 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/534/dartdoc July 25, 2024 15:52 Inactive
@maratal maratal force-pushed the fix/533-dont-request-push-token branch from 71b24d0 to 4709101 Compare July 25, 2024 21:42
@github-actions github-actions bot temporarily deployed to staging/pull/534/dartdoc July 25, 2024 21:43 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/534/features July 25, 2024 21:43 Inactive
@maratal maratal force-pushed the fix/533-dont-request-push-token branch from 4709101 to cbeaea0 Compare July 28, 2024 16:14
@github-actions github-actions bot temporarily deployed to staging/pull/534/dartdoc July 28, 2024 16:15 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/534/features July 28, 2024 16:15 Inactive
…ocoa and received by flutter, which sends it to ably via the first available `ARTRealtime` object. All instances have the same value for the `device.push` property, since the `device` itself is stored in a static variable.
@maratal maratal force-pushed the fix/533-dont-request-push-token branch from cbeaea0 to 25ef834 Compare July 28, 2024 18:27
@github-actions github-actions bot temporarily deployed to staging/pull/534/features July 28, 2024 18:28 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/534/dartdoc July 28, 2024 18:28 Inactive
ios/Classes/AblyInstanceStore.m Outdated Show resolved Hide resolved
ios/Classes/AblyFlutter.m Show resolved Hide resolved
…moteNotificationsWithDeviceToken` on a proper instance of the realtime/rest that called push activation.
@github-actions github-actions bot temporarily deployed to staging/pull/534/features August 10, 2024 15:22 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/534/dartdoc August 10, 2024 15:22 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Library doesn't respect RSH3a3a
2 participants