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

[WIP]Feature/live modeling #230

Closed
wants to merge 2 commits into from
Closed

[WIP]Feature/live modeling #230

wants to merge 2 commits into from

Conversation

bjoernmu
Copy link

@bjoernmu bjoernmu commented Feb 19, 2021

This pull request is to prepare the merge of #204

DONE:
-merge master
-fix errors that prevent start
-fix errors
-connect to Container

TODO:
-check semantic
-check plan generation in Container
-drag and drop of new nodes does not seem to work properly

In YAML Mode:
-Properties are shown only in sidebar not in dropdown of the nodetemplate
-Error on Save: ERROR o.e.w.r.rest.websockets.GitWebSocket:280 run - org.eclipse.jgit.diff.DiffEntry cannot be cast to java.lang.String java.lang.ClassCastException: org.eclipse.jgit.diff.DiffEntry cannot be cast to java.lang.String at org.eclipse.winery.repository.rest.websockets.GitWebSocket$GitLogRunnable.run(GitWebSocket.java:253)
-"Hide DependsOn Relations" Button routes to http://localhost:4201/

@nyuuyn
Copy link

nyuuyn commented May 11, 2021

Everything except property validation should work right now

@nyuuyn
Copy link

nyuuyn commented May 14, 2021

Adding a small video for reviewers to look into the new feature to have the following changes ready in the runtime: OpenTOSCA/container#190

https://youtu.be/cJKXwwdNfrw

Co-authored-by: Kálmán Képes <kalman.kepes@iaas.uni-stuttgart.de>
Co-authored-by: Björn Müller <bjoern.mueller@kanu-baerchen.de>
Co-authored-by: Lukas Harzenetter <lharzenetter@gmx.de>
Signed-off-by: Phi Dang <dngphg@gmail.com>
Signed-off-by: Lukas Harzenetter <lharzenetter@gmx.de>
Signed-off-by: Lukas Harzenetter <lharzenetter@gmx.de>
@lharzenetter
Copy link
Member

Follow up at eclipse.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants