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

refactor!: drop update() method support #2825

Merged
merged 1 commit into from
Oct 11, 2021

Conversation

vursen
Copy link
Contributor

@vursen vursen commented Oct 11, 2021

Description

The PR drops support for the update() method that has been deprecated since Vaadin 21.

Fixes #2779

Type of change

  • Breaking-change

Checklist

  • I have read the contribution guide: https://vaadin.com/docs-beta/latest/guide/contributing/overview/
  • I have added a description following the guideline.
  • The issue is created in the corresponding repository and I have referenced it.
  • I have added tests to ensure my change is effective and works as intended.
  • New and existing tests are passing locally with my change.
  • I have performed self-review and corrected misspellings.

@sonarcloud
Copy link

sonarcloud bot commented Oct 11, 2021

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

@web-padawan web-padawan removed the request for review from tomivirkki October 11, 2021 14:42
@web-padawan web-padawan merged commit 0eb9b1f into master Oct 11, 2021
@web-padawan web-padawan deleted the refactor/drop-update-method branch October 11, 2021 14:42
@vaadin-bot
Copy link
Collaborator

This ticket/PR has been released with platform 22.0.0.

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.

Drop render method from components that have it
3 participants