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

Create CustomerCenterListener #2199

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Conversation

vegaro
Copy link
Contributor

@vegaro vegaro commented Feb 26, 2025

This will allow developers to listen to events that occur inside the Customer Center

@vegaro
Copy link
Contributor Author

vegaro commented Feb 26, 2025

@RevenueCat/coresdk can you please take a look at this draft, I would like to make sure I get the public stuff right before continuing implementing.

Copy link
Contributor

@tonidero tonidero left a comment

Choose a reason for hiding this comment

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

I wonder if there is anything we can do to make the interface in kotlin... Probably not the end of the world though. Looking good!

* The listener is responsible for handling actions that happened in the Customer Center.
* Make sure [removeCustomerCenterListener] is called when the listener needs to be destroyed.
*/
var customerCenterListener: CustomerCenterListener?
Copy link
Contributor

Choose a reason for hiding this comment

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

I wonder if we would want to support multiple listeners... But then again, this is the API we have for the CustomerInfo, so it might make more sense to be consistent 👍

Copy link

emerge-tools bot commented Feb 27, 2025

📸 Snapshot Test

260 unchanged

Name Added Removed Modified Renamed Unchanged Errored Approval
TestPurchasesUIAndroidCompatibility
com.revenuecat.testpurchasesuiandroidcompatibility
0 0 0 0 260 0 N/A

🛸 Powered by Emerge Tools

@vegaro vegaro force-pushed the cc-216-callback-with-actions branch from 5fc0c06 to 0bce4b8 Compare February 28, 2025 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants