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

feat: enable components to notify binder when validation state changes (#13940) (CP: 9.0) #13996

Merged
merged 3 commits into from Jun 20, 2022

Conversation

vaadin-bot
Copy link
Collaborator

@vaadin-bot vaadin-bot commented Jun 16, 2022

Adds the needed API to notify Binder about validation status changes that happen in field components.

Fixes #8242
Related to vaadin/flow-components#1158

#13940)

* feat: enable components to notify binder about their validation status changes

Co-authored-by: Mikhail Shabarov <61410877+mshabarov@users.noreply.github.com>
@sonarcloud
Copy link

sonarcloud bot commented Jun 20, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@taefi taefi enabled auto-merge (squash) June 20, 2022 07:49
@taefi taefi added +0.1.0 and removed +1.0.0 labels Jun 20, 2022
@github-actions
Copy link

github-actions bot commented Jun 20, 2022

Unit Test Results

   770 files  +1     770 suites  +1   25m 13s ⏱️ -23s
5 735 tests +8  5 683 ✔️ +8  52 💤 ±0  0 ±0 
5 780 runs  +5  5 727 ✔️ +5  53 💤 ±0  0 ±0 

Results for commit b6c8c7f. ± Comparison against base commit 391c788.

♻️ This comment has been updated with latest results.

@taefi taefi merged commit 1706607 into 9.0 Jun 20, 2022
@taefi taefi deleted the cherry-pick-13940-to-9.0-1655377061530 branch June 20, 2022 10:01
@vaadin-bot
Copy link
Collaborator Author

This ticket/PR has been released with Vaadin 22.0.18.

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

Successfully merging this pull request may close these issues.

None yet

3 participants