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

Async event handlers #1406

Open
MisinformedDNA opened this issue Jun 21, 2022 · 1 comment
Open

Async event handlers #1406

MisinformedDNA opened this issue Jun 21, 2022 · 1 comment
Assignees
Labels
doc-enhancement Improve the current content [org][type][category] dotnet-desktop/svc Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Label to indicate an item has been imported. winforms/subsvc

Comments

@MisinformedDNA
Copy link

MisinformedDNA commented Jun 21, 2022

Type of issue

Other (describe below)

Description

There should be documentation about how async event handlers work, best practices and pitfalls.

One issue is this documentation from DevExpress that instructs developers to use .GetAwaiter().GetResult();. This which would likely cause deadlocks, and I wanted to point DevExpress to documentation that shows them why this is bad, but cannot find anything official on why it is bad or what the workarounds may be.

There are countless more SO posts about various async issues when using WinForms.

Added by adegeo:
Good reference is at https://devblogs.microsoft.com/dotnet/configureawait-faq/

Page URL

https://learn.microsoft.com/en-us/dotnet/desktop/winforms/forms/events?view=netdesktop-6.0

Content source URL

https://github.com/dotnet/docs-desktop/blob/main/dotnet-desktop-guide/net/winforms/forms/events.md

Document Version Independent Id

4d5076d1-338f-1d67-c72a-09e71f62c15c

Article author

@adegeo


Associated WorkItem - 291113

@dotnet-bot dotnet-bot added the ⌚ Not Triaged Not triaged label Jun 21, 2022
@PRMerger20 PRMerger20 added dotnet-desktop/svc winforms/subsvc Pri1 High priority, do before Pri2 and Pri3 labels Jun 21, 2022
@adegeo
Copy link
Contributor

adegeo commented Jun 27, 2022

Thank you for this feedback! We'll look at getting this issue prioritized.

@adegeo adegeo added the doc-enhancement Improve the current content [org][type][category] label Jun 27, 2022
@adegeo adegeo self-assigned this Jun 27, 2022
@dotnet-bot dotnet-bot removed the ⌚ Not Triaged Not triaged label Jun 27, 2022
@adegeo adegeo added the 🗺️ reQUEST Trigger label to import an issue into Quest label Jun 13, 2024
@sequestor sequestor bot added 📌 seQUESTered Label to indicate an item has been imported. and removed 🗺️ reQUEST Trigger label to import an issue into Quest labels Jun 14, 2024
@adegeo adegeo moved this from 🔖 Ready to Slipped in dotnet/docs July 2024 Sprint Jul 29, 2024
@adegeo adegeo added the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Aug 1, 2024
@dotnet-bot dotnet-bot removed the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Aug 1, 2024
@adegeo adegeo added the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Aug 4, 2024
@dotnet-bot dotnet-bot removed the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Aug 4, 2024
@adegeo adegeo added the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Aug 4, 2024
@dotnet-bot dotnet-bot removed the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Aug 4, 2024
@dotnetrepoman dotnetrepoman bot added 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. and removed 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. labels Nov 5, 2024
@adegeo adegeo moved this from 🔖 Ready to Slipped in dotnet/docs October 2024 Sprint Nov 5, 2024
@dotnetrepoman dotnetrepoman bot removed the 🗺️ mapQUEST Only used as a way to mark an issue as updated. RepoMan should instantly remove it. label Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc-enhancement Improve the current content [org][type][category] dotnet-desktop/svc Pri1 High priority, do before Pri2 and Pri3 📌 seQUESTered Label to indicate an item has been imported. winforms/subsvc
Projects
Status: 🔖 Ready
Status: Slipped
Status: Slipped
Development

No branches or pull requests

4 participants