-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: public api for dismiss message #420
Conversation
Sample app builds 📱Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.
|
Build available to test |
📏 SDK Binary Size Comparison ReportNo changes detected in SDK binary size ✅ |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #420 +/- ##
============================================
+ Coverage 41.98% 42.00% +0.01%
Complexity 259 259
============================================
Files 99 99
Lines 2320 2319 -1
Branches 344 343 -1
============================================
Hits 974 974
+ Misses 1247 1246 -1
Partials 99 99 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
da65b77
to
e454484
Compare
## [4.1.0](4.0.1...4.1.0) (2024-08-09) ### Features * public api for dismiss message ([#420](#420)) ([8bfc37d](8bfc37d))
Background
Since we decoupled all modules from
CustomerIO
class and moved them tocore
module, modules were moved to theSDKComponent
class. This broke some public APIs accessed throughSDKComponent
and made some of our documentation outdated (e.g., dismissing in-app messaging). To fix this, we had two options:Option 1 - Expose in-app module through
CustomerIO
object again.I think this is the most feasible option, so I opted for this.
Option 2 - Ask customers to use
SDKComponent
instead.I don't think recommending customers to use
SDKComponent
is a very good idea, so I opted for option 1.Changes
CustomerIO
object for customer easeSDKComponent
methods exposing these modules as internalResources