ably-cocoa icon indicating copy to clipboard operation
ably-cocoa copied to clipboard

Refactor presence implementation and re-enable skipped presence tests (umbrella issue).

Open maratal opened this issue 2 years ago • 2 comments

Also, I would love someone to go through the current presence implementation and check if it's compatible with spec https://sdk.ably.com/builds/ably/specification/main/features/#RTP1. You can refer to https://github.com/ably/ably-go/blob/14cd5a206e18571f1012f992397e1092c535919e/ably/realtime_presence.go file for more refined implementation in accordance with spec.

Originally posted by sacOO7 in https://github.com/ably/ably-cocoa/pull/1714#discussion_r1423840044

Umbrella issue for:

  • [x] https://github.com/ably/ably-cocoa/issues/1875
  • [x] https://github.com/ably/ably-cocoa/issues/1876
  • [x] https://github.com/ably/ably-cocoa/issues/1890
  • [ ] https://github.com/ably/ably-cocoa/issues/1889

┆Issue is synchronized with this Jira Task by Unito

maratal avatar Dec 22 '23 16:12 maratal

➤ Automation for Jira commented:

The link to the corresponding Jira issue is https://ably.atlassian.net/browse/SDK-4009

sync-by-unito[bot] avatar Dec 22 '23 16:12 sync-by-unito[bot]

As a part of refactor, we also need to move presence code from channel to separate presence file

sacOO7 avatar Jan 12 '24 16:01 sacOO7