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

[core] remove virtual-widget and virtual-renderer #6885

Merged
merged 1 commit into from
Jan 15, 2020
Merged

Conversation

vince-fugnitto
Copy link
Member

@vince-fugnitto vince-fugnitto commented Jan 14, 2020

What it does

Fixes #2300

Both the virtual-widget and virtual-renderer have been deprecated since #2301. This patch cleans up the codebase to remove the deprecated code as it should no longer be used/supported.

  • removes the deprecated virtual-widget.
  • removes the deprecated virtual-renderer.

One should instead opt to use react-widget and react-renderer.

How to test

Verify that CI successfully passes (build & tests)

Review checklist

Reminder for reviewers

@vince-fugnitto vince-fugnitto added the quality issues related to code and application quality label Jan 14, 2020
@vince-fugnitto vince-fugnitto self-assigned this Jan 14, 2020
@akosyakov
Copy link
Member

Please add a word about it in the CHANGELOG.

@akosyakov akosyakov added the shell issues related to the core shell label Jan 15, 2020
Fixes #2300

Both the `virtual-widget` and `virtual-renderer` have been deprecated
since #2301. This patch cleans
up the codebase to remove the deprecated code as it should no longer be used/supported.

- removes the deprecated `virtual-widget`.
- removes the deprecated `virtual-renderer`.

One should instead opt to use `react-widget` and `react-renderer`.

Signed-off-by: vince-fugnitto <vincent.fugnitto@ericsson.com>
@vince-fugnitto
Copy link
Member Author

Please add a word about it in the CHANGELOG.

Thanks! I updated it to include changelog entries.

Copy link
Member

@akosyakov akosyakov left a comment

Choose a reason for hiding this comment

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

thank you!

@vince-fugnitto vince-fugnitto merged commit 868b58c into master Jan 15, 2020
@vince-fugnitto vince-fugnitto deleted the vf/GH-2300 branch January 15, 2020 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quality issues related to code and application quality shell issues related to the core shell
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove VirtualWidget and VirtualRenderer with next major release
2 participants