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

Provide a direct means to observe changes in ApolloStore #300

Merged

Conversation

jamesonwilliams
Copy link
Contributor

(Full context in apollographql/apollo-ios#3350.)

Apollo Kotlin currently provides a public changedKeys property to get notified when objects in the ApolloStore change. Internally, iOS has had a similar facility, too -- but it hasn't been public until now.

@jamesonwilliams jamesonwilliams requested a review from a team as a code owner March 15, 2024 04:36
Copy link

netlify bot commented Mar 15, 2024

👷 Deploy request for apollo-ios-docc pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 6f66f7b

Copy link

netlify bot commented Mar 15, 2024

👷 Deploy request for eclectic-pie-88a2ba pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit 6f66f7b

@AnthonyMDev AnthonyMDev self-assigned this Mar 21, 2024
Copy link
Contributor

@AnthonyMDev AnthonyMDev left a comment

Choose a reason for hiding this comment

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

Code for this looks good! Just one request on the tests and then I'm good to merge this. Thanks so much for the PR!

Tests/ApolloTests/Cache/StoreSubscriptionTests.swift Outdated Show resolved Hide resolved
Apollo Kotlin currently provides a public API to get notified when
objects in the ApolloStore are updated. iOS has a similar facility as
well, but it has not been a public API. This change exposes the
ApolloStoreSubscriber for public use.

Refer: apollographql/apollo-ios#3350
@jamesonwilliams jamesonwilliams force-pushed the public-apollo-store-subscriber branch 2 times, most recently from e31d708 to f3f6f21 Compare March 22, 2024 00:13
@AnthonyMDev AnthonyMDev merged commit 0e87a8f into apollographql:main Mar 22, 2024
14 checks passed
BobaFetters pushed a commit to apollographql/apollo-ios that referenced this pull request Mar 22, 2024
BobaFetters pushed a commit that referenced this pull request Mar 22, 2024
3a70e655 Provide a direct means to observe changes in ApolloStore (#300)

git-subtree-dir: apollo-ios
git-subtree-split: 3a70e6557f1cd97879ee76dbe5b6f39c466f770e
BobaFetters pushed a commit that referenced this pull request Mar 22, 2024
…es in ApolloStore

git-subtree-dir: apollo-ios
git-subtree-mainline: 2b6b61b
git-subtree-split: 3a70e6557f1cd97879ee76dbe5b6f39c466f770e
@jamesonwilliams jamesonwilliams deleted the public-apollo-store-subscriber branch March 22, 2024 01:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants