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: Make findAncestor return Object for backward compatibility #14242

Merged
merged 1 commit into from Aug 2, 2022

Conversation

mshabarov
Copy link
Contributor

@mshabarov mshabarov commented Aug 1, 2022

Changes return type to be compatible with classic components package API. Not a breaking change since this method was added recently for 23.2.

Related-to https://github.com/vaadin/classic-components-internal/pull/164

@sonarcloud
Copy link

sonarcloud bot commented Aug 1, 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

@github-actions
Copy link

github-actions bot commented Aug 1, 2022

Unit Test Results

   905 files  ±0     905 suites  ±0   50m 13s ⏱️ - 6m 30s
5 896 tests ±0  5 853 ✔️ ±0  43 💤 ±0  0 ±0 
6 121 runs  +3  6 071 ✔️ +3  50 💤 ±0  0 ±0 

Results for commit 82b35c8. ± Comparison against base commit e44a7eb.

@caalador caalador merged commit e084d39 into master Aug 2, 2022
OLD Vaadin Flow ongoing work (Vaadin 10+) automation moved this from Iteration Reviews to Done - pending release Aug 2, 2022
@caalador caalador deleted the findancestor-return-object branch August 2, 2022 05:02
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with Vaadin 23.2.0.beta1 and is also targeting the upcoming stable 23.2.0 version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
OLD Vaadin Flow ongoing work (Vaadin ...
  
Done - pending release
Development

Successfully merging this pull request may close these issues.

None yet

3 participants