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

Remove the DevWorkspace handler code from the Che-Code repository #21837

Closed
azatsarynnyy opened this issue Nov 24, 2022 · 3 comments
Closed

Remove the DevWorkspace handler code from the Che-Code repository #21837

azatsarynnyy opened this issue Nov 24, 2022 · 3 comments
Assignees
Labels
area/editor/vscode Issues related to the Code OSS editor of Che kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system. sprint/current team/B This team is responsible for the Web Terminal, the DevWorkspace Operator and the IDEs.
Milestone

Comments

@azatsarynnyy
Copy link
Member

azatsarynnyy commented Nov 24, 2022

Is your task related to a problem? Please describe

Once we are done with the migration to DevWorkspace Operator container contributions we can remove the DevWorkspace handler-related code from the Che-Code repository as it will become not used anymore.

Describe the solution you'd like

  1. Make sure there are no consumers of the eclipse-che/che-code-devworkspace-handler npm library.
    Currently, the known consumers are:
  1. Remove the GitHub Workflow publishing the new versions of the npm package to the public npmjs registry.
  2. Remove the devworkspace-handler code.

Describe alternatives you've considered

No response

Additional context

The issue is a part of epic #21736

@azatsarynnyy azatsarynnyy added kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system. team/editors area/editor/vscode Issues related to the Code OSS editor of Che labels Nov 24, 2022
@ibuziuk
Copy link
Member

ibuziuk commented Jan 20, 2023

@l0rd @azatsarynnyy could you please prioritize it for the next sprint to finalize the epic - #21736

@azatsarynnyy
Copy link
Member Author

Before this clean-up, we should wait for this handler usage to be removed from the Dashboard. We've already discussed that with @ibuziuk and @olexii4
The Dashboard part is tracked at #21738

@ibuziuk ibuziuk added the team/B This team is responsible for the Web Terminal, the DevWorkspace Operator and the IDEs. label Jan 25, 2023
@azatsarynnyy azatsarynnyy modified the milestones: 7.61, 7.62 Feb 16, 2023
@l0rd l0rd mentioned this issue Mar 6, 2023
7 tasks
@azatsarynnyy
Copy link
Member Author

The DevWorkspace handler dependency is removed on the Dashboard side.

@vitaliy-guliy I think we can merge the Che-Code PR che-incubator/che-code#169.

@azatsarynnyy azatsarynnyy modified the milestones: 7.62, 7.63 Mar 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/editor/vscode Issues related to the Code OSS editor of Che kind/task Internal things, technical debt, and to-do tasks to be performed. severity/P2 Has a minor but important impact to the usage or development of the system. sprint/current team/B This team is responsible for the Web Terminal, the DevWorkspace Operator and the IDEs.
Projects
None yet
Development

No branches or pull requests

5 participants