RIDGIDSoftwareSolutions / Universal-Analytics-For-DotNet

A .NET wrapper over top of Google's Universal Analytics Measurement Protocol API
MIT License
14 stars 5 forks source link

Add support for sending analytics asynchronously #6

Closed david2h1 closed 7 years ago

jakejgordon commented 7 years ago

Hi @david2h1 -- any thoughts on my previous comment?

jakejgordon commented 7 years ago

bump @david2h1

david2h1 commented 7 years ago

-- apologies for the delay in getting back to you, @jakejgordon.

Awaiting the TrackEventAsync method is very useful for the devs using your library. The caller of this method (i.e. the dev's enclosing method) will continue its execution without being I/O bound; the TrackEventAsync task, however, still runs within the same context.

If instead you implement this as a fire-and-forget without awaiting, you also deprive the dev of the option to handle failures.

HTH, David