Skip to content

fix: Gateway plugin should list the workspaces created from a custom editor definition#189

Merged
adietish merged 1 commit intoredhat-developer:mainfrom
vrubezhny:fix-custom-workspaces-not-listed
Oct 16, 2025
Merged

fix: Gateway plugin should list the workspaces created from a custom editor definition#189
adietish merged 1 commit intoredhat-developer:mainfrom
vrubezhny:fix-custom-workspaces-not-listed

Conversation

@vrubezhny
Copy link
Collaborator

The DevWorkspaceTemplate object is used to detect if the JetBrains Idea based editor was used to create a WS

Fixes: eclipse-che/che#23610

@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2025

Codecov Report

❌ Patch coverage is 0% with 105 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (71098f6) to head (8b017dc).
⚠️ Report is 234 commits behind head on main.

Files with missing lines Patch % Lines
...devtools/gateway/openshift/DevWorkspaceTemplate.kt 0.00% 62 Missing ⚠️
...redhat/devtools/gateway/openshift/DevWorkspaces.kt 0.00% 32 Missing ⚠️
.../redhat/devtools/gateway/openshift/DevWorkspace.kt 0.00% 9 Missing ⚠️
...lin/com/redhat/devtools/gateway/openshift/Utils.kt 0.00% 1 Missing ⚠️
.../gateway/view/steps/DevSpacesWorkspacesStepView.kt 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            main    #189     +/-   ##
=======================================
  Coverage   0.00%   0.00%             
=======================================
  Files          4      32     +28     
  Lines         26    1151   +1125     
  Branches       0     199    +199     
=======================================
- Misses        26    1151   +1125     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Member

@azatsarynnyy azatsarynnyy left a comment

Choose a reason for hiding this comment

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

I tried to connect to the dogfooding instance:
Image

@vrubezhny
Copy link
Collaborator Author

I tried to connect to the dogfooding instance:

Any chance to see the stacktrace?

@vrubezhny vrubezhny force-pushed the fix-custom-workspaces-not-listed branch 2 times, most recently from 65eea6e to bef35e3 Compare October 15, 2025 11:06
Copy link
Collaborator

@adietish adietish left a comment

Choose a reason for hiding this comment

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

LGTM.
Tested it with the custom editor definition that was causing an error for @azatsarynnyy. Worked well for me.

@vrubezhny vrubezhny force-pushed the fix-custom-workspaces-not-listed branch from bef35e3 to ac85fa5 Compare October 15, 2025 15:28
@vrubezhny vrubezhny force-pushed the fix-custom-workspaces-not-listed branch from ac85fa5 to a759101 Compare October 15, 2025 17:39
@vrubezhny vrubezhny marked this pull request as draft October 15, 2025 17:40
@vrubezhny vrubezhny force-pushed the fix-custom-workspaces-not-listed branch from a759101 to f005d7c Compare October 15, 2025 18:11
…editor definition

The DevWorkspaceTemplate object is used to detect if the JetBrains Idea based editor was used to create a WS

Fixes: eclipse-che/che#23610

Signed-off-by: Victor Rubezhny <vrubezhny@redhat.com>
@vrubezhny vrubezhny force-pushed the fix-custom-workspaces-not-listed branch from f005d7c to 8b017dc Compare October 15, 2025 19:02
@vrubezhny vrubezhny requested a review from adietish October 15, 2025 19:07
@vrubezhny vrubezhny marked this pull request as ready for review October 15, 2025 19:10
Copy link
Member

@azatsarynnyy azatsarynnyy left a comment

Choose a reason for hiding this comment

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

👍
Image

Copy link
Collaborator

@adietish adietish left a comment

Choose a reason for hiding this comment

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

works great for me. LGTM

@adietish adietish merged commit 09f7223 into redhat-developer:main Oct 16, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Gateway plugin should list the workspaces created from a custom editor definition

4 participants