klaviyo / klaviyo-swift-sdk

Swift SDK for integrating Klaviyo’s push notifications, event tracking, and user profile management into iOS applications.

https://www.klaviyo.com/mobile-push-marketing

MIT License

12 stars 11 forks source link

Flush requests queue on opened push event #183

Closed ajaysubra closed 3 months ago

ajaysubra commented 3 months ago

Description

In order to keep engagement metric as up to date as possible we want to try and flush the queue whenever an opened push event request is enqueued.

Check List

[ ] Are you changing anything with the public API?
[ ] Have you tested this change on real device?
[ ] Are your changes backwards compatible with previous SDK Versions?
[ ] Have you added unit test coverage for your changes?
[ ] Have you verified that your changes are compatible with all the operating system version this SDK currently supports?

Manual Test Plan

Unit tests but also using a proxy -

Increased the flush interval to something very high (100 secs)
Enqueued various events and confirmed they didn't flush
Enqueued a opened push event and noticed in the proxy that they all flushed.