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

Expose a SendNotificationAsync Method #8

Closed
nikmd23 opened this issue Feb 22, 2017 · 4 comments
Closed

Expose a SendNotificationAsync Method #8

nikmd23 opened this issue Feb 22, 2017 · 4 comments
Assignees
Milestone

Comments

@nikmd23
Copy link

nikmd23 commented Feb 22, 2017

It'd be great if we could send notifications asynchronously.

It looks like there's an explicit .Wait() call right now.

@coryjthompson
Copy link
Member

Agreed. I'll try and find some time this week to learn how to do ASYNC correctly pre .NET v4.5.

I believe Microsoft.Bcl.Async is my way forwards while maintaining backwards compatibility and mono support.

@coryjthompson coryjthompson added this to the 1.0.9 milestone Mar 6, 2017
@nikmd23
Copy link
Author

nikmd23 commented Mar 6, 2017

Awesome!

I didn't think about pre .NET 4.5 support. I just assumed that there'd be an additional async method added for newer versions of .NET, but what you're thinking about could be promising. I'm looking forward to it.

@marsimoto03
Copy link

Hi,
any updates when this feature will be available and published as new version on nuget?
Thank you!

@coryjthompson
Copy link
Member

Hi,

Sorry work ramped up quite a bit. I'm hoping to find some time this weekend.

Once this feature is closed I will release on NuGet.

:)

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

No branches or pull requests

3 participants