Skip to content

Implement new readonly (getter only) property of ErrorProvider.HasErrors - #6234

Merged
dreddy-work merged 7 commits into
dotnet:mainfrom
eriawan:implement-errorprovider-haserrors
Dec 15, 2021
Merged

Implement new readonly (getter only) property of ErrorProvider.HasErrors#6234
dreddy-work merged 7 commits into
dotnet:mainfrom
eriawan:implement-errorprovider-haserrors

Conversation

@eriawan

@eriawan eriawan commented Nov 26, 2021

Copy link
Copy Markdown
Member

Fixes #5912

Proposed changes

  • No change, it is a new readonly property to check controls in an ErrorProvider are having errors set (true) or not

Customer Impact

Regression?

  • No

Risk

  • Should be none, because it is not a breaking change

Screenshots

Before

After

Test methodology

Accessibility testing

Test environment(s)

Microsoft Reviewers: Open in CodeFlow
@eriawan
eriawan requested a review from a team as a code owner November 26, 2021 00:54
@ghost ghost assigned eriawan Nov 26, 2021
@eriawan eriawan changed the title Implement new readobly (getter only) property of ErrorProvider.HasErrors Nov 26, 2021
@eriawan

eriawan commented Nov 27, 2021

Copy link
Copy Markdown
Member Author

@RussKie
Please review, thanks in advance 🙂

@RussKie RussKie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Please add tests

@RussKie RussKie added the waiting-author-feedback The team requires more information from the author label Nov 29, 2021
@eriawan

eriawan commented Nov 29, 2021

Copy link
Copy Markdown
Member Author

@RussKie
thanks!

I'm going to add unit tests for this new HasErrors property.

@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Nov 29, 2021
@eriawan

eriawan commented Nov 30, 2021

Copy link
Copy Markdown
Member Author

@RussKie

unit tests added. please review, thanks in advance 🙂

Comment thread src/System.Windows.Forms/src/System/Windows/Forms/ErrorProvider.cs Outdated

@RussKie RussKie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost there

@ghost ghost added the waiting-author-feedback The team requires more information from the author label Dec 1, 2021
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Dec 1, 2021

@RussKie RussKie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Comment thread src/System.Windows.Forms/src/System/Windows/Forms/ErrorProvider.cs Outdated
…r.cs

Co-authored-by: Genevieve Warren <24882762+gewarren@users.noreply.github.com>
RussKie
RussKie previously approved these changes Dec 2, 2021
@eriawan

eriawan commented Dec 3, 2021

Copy link
Copy Markdown
Member Author

@RussKie
Thanks for the update on the XML summary comment 🙂
When this PR be merged?
Is there anything else I have to update?

@RussKie
RussKie requested a review from dreddy-work December 6, 2021 00:25
@RussKie

RussKie commented Dec 6, 2021

Copy link
Copy Markdown
Contributor

@eriawan - no, I don't think so. @dreddy-work will review and merge this PR, once he's ready.

@RussKie RussKie added the waiting-on-team This work item needs to be discussed with team or is waiting on team action in order to proceed label Dec 9, 2021
Comment thread src/System.Windows.Forms/src/PublicAPI.Shipped.txt Outdated
@dreddy-work dreddy-work added waiting-author-feedback The team requires more information from the author and removed waiting-on-team This work item needs to be discussed with team or is waiting on team action in order to proceed labels Dec 13, 2021
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Dec 14, 2021
@dreddy-work dreddy-work added the waiting-author-feedback The team requires more information from the author label Dec 14, 2021
@ghost ghost removed the waiting-author-feedback The team requires more information from the author label Dec 15, 2021
@eriawan
eriawan requested a review from dreddy-work December 15, 2021 18:02
@dreddy-work
dreddy-work merged commit 6dfe634 into dotnet:main Dec 15, 2021
@ghost ghost added this to the 7.0 Preview1 milestone Dec 15, 2021
@eriawan

eriawan commented Dec 15, 2021

Copy link
Copy Markdown
Member Author

Thanks!

@eriawan
eriawan deleted the implement-errorprovider-haserrors branch December 15, 2021 18:20
@ghost ghost locked as resolved and limited conversation to collaborators Jan 25, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

4 participants