Skip to content
This repository was archived by the owner on Sep 29, 2023. It is now read-only.

Conversation

abnegate
Copy link
Member

No description provided.

@abnegate abnegate requested a review from eldadfux February 13, 2022 23:39
@@ -167,12 +194,24 @@ subscription.stream.listen((response) {
<li>
<h3>Android</h3>
<div class="ide margin-bottom" data-lang="android" data-lang-label="Android SDK">
<pre class="line-numbers"><code class="prism language-kotlin" data-prism>val client = Client()
<pre class="line-numbers"><code class="prism language-kotlin" data-prism>val client = Client(context)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

have we updated everywhere that client requires context in android?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep 👍

Copy link
Member

@lohanidamodar lohanidamodar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@abnegate rest looks good to me, just verify that we have not missed anywhere that android client requires context.

@lohanidamodar lohanidamodar merged commit 4bfa9b5 into main Feb 15, 2022
@lohanidamodar lohanidamodar deleted the feat-realtime-updates branch February 15, 2022 01:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants