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

fix: do not warn for unset disabled property change (#13509) (CP: 2.8) #13515

Merged
merged 1 commit into from Apr 13, 2022

Conversation

mcollovati
Copy link
Collaborator

No description provided.

Do not log a waning if getting a
property change for a disabled
component when that property has
not been set for the component.

Fixes vaadin/flow-components#2490
@sonarcloud
Copy link

sonarcloud bot commented Apr 12, 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
No Duplication information No Duplication information

@caalador caalador merged commit 4e454fd into 2.8 Apr 13, 2022
@caalador caalador deleted the cherry-pick-13509-to-2.8 branch April 13, 2022 04:13
vaadin-bot pushed a commit that referenced this pull request Apr 13, 2022
Do not log a waning if getting a
property change for a disabled
component when that property has
not been set for the component.

Fixes vaadin/flow-components#2490

Co-authored-by: caalador <mikael.grankvist@vaadin.com>
mcollovati added a commit that referenced this pull request Apr 13, 2022
Do not log a waning if getting a
property change for a disabled
component when that property has
not been set for the component.

Fixes vaadin/flow-components#2490

Co-authored-by: caalador <mikael.grankvist@vaadin.com>
vaadin-bot added a commit that referenced this pull request Apr 13, 2022
…#13519)

Do not log a waning if getting a
property change for a disabled
component when that property has
not been set for the component.

Fixes vaadin/flow-components#2490

Co-authored-by: caalador <mikael.grankvist@vaadin.com>

Co-authored-by: Marco Collovati <marco@vaadin.com>
Co-authored-by: caalador <mikael.grankvist@vaadin.com>
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