-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 samples endpoint issues #13964
Comments
@l0rd if you do not mind I will put this issue to the End Game Plan as a subtask for |
@ibuziuk of course that's perfect. One thing though: this can also be a multi-team effort as it has been the case for #13529 (comment). You coordinate that but others teams can help as well. I think it worked pretty well for #13529 |
From what I've seen, the current endpoint issues are:
|
I've opened PR eclipse-che/che-devfile-registry#77 to work around the php devfile issues. The NodeJS + Mongo devfile is (I think) just a bad sample application; it's hard to interact with a REST backend from the browser. |
@slemeur @l0rd, in general, I believe we need to reconsider the samples we are using and get rid of / deprecate https://github.com/che-samples . IMO we should use vendor samples for all the devfiles like https://github.com/spring-projects/spring-petclinic |
Closing:
|
Is your enhancement related to a problem? Please describe
For some of the samples projects in the devfile registry, the preview URL proposed by Theia port plugin is not the correct application URL. Even if the workspace sample starts properly users won't be able to open the application in the preview panel. That's definitely a bad UX.
Describe the solution you'd like
The short term solution is to:
The long term solution is to add a new previewURL attribute to the devfiles commands #13945
The text was updated successfully, but these errors were encountered: