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

Add a completion handler to TelemetryManager.send() #127

Open
mjalq opened this issue Nov 3, 2023 · 0 comments
Open

Add a completion handler to TelemetryManager.send() #127

mjalq opened this issue Nov 3, 2023 · 0 comments

Comments

@mjalq
Copy link

mjalq commented Nov 3, 2023

A simple escaping completion handler would suffice.

This would be useful if I want the app to send one signal per day at most, while making sure that the signal was received by the server before blocking further attempts.

`

    if lastSignalSentDate != today {
        TelemetryManager.send("appUsed")
        //If the above line fails due to poor internet or an offline launch of the app, we have no way of knowing.
        lastSignalSentDate = today
        
    }

`

Purposed solution

`

    if lastSignalSentDate != today {
        TelemetryManager.send("appUsed") { didFail in
            
            if !didFail {
                lastSignalSentDate = today

            }
        }
        
    }

`

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant