tobiashochguertel / c4-draw.io

C4 Modelling little bit easier
https://tobiashochguertel.github.io/c4-draw.io/
GNU General Public License v3.0
794 stars 105 forks source link

C4 properties do not show up on draw.io desktop 13.0.1 #19

Open elsni opened 4 years ago

elsni commented 4 years ago

I managed to install the plugin and activated the C4 shape menu which was hidden by default. When I press ctrl-M on a C4 shape, the properties are blank.

thilanka02 commented 4 years ago

I'm having the same issue. Suddenly C4 plugin stopped working and removed from the shapes. There is an issue, loading the script. image

chathuranga commented 4 years ago

Hi. Any update on this?

tobiashochguertel commented 4 years ago

I had an issue with the drawio desktop development env, and asked them to have a look: https://github.com/jgraph/drawio-desktop/issues/74

I found the same issue already closed and added my question there.

I will later or tomorrow try to reproduce this issue later on my new computer which runs CentOS. I‘ve got this issue as I was working on a Mac OS.

tobiashochguertel commented 4 years ago

With the current version of draw.io Desktop 13.2.2 (build from source https://github.com/jgraph/drawio-desktop ) it working again?

image

I see that they had modified something with the plugin dialog, maybe now it's workimng with an more current version?

@elsni can you verify it?

mofe23 commented 4 years ago

Works for me on draw.io desktop 13.2.4 installed today. Thx for your work tobi!

chathuranga commented 4 years ago

Any idea of getting the same "Content Security Policy" issue fixed in the Draw.io plugin version?

tobiashochguertel commented 4 years ago

Any idea of getting the same "Content Security Policy" issue fixed in the Draw.io plugin version?

@chathuranga Do you have this issue with the http://draw.io Variant and adding C4 Plugin via "https://tobiashochguertel.github.io/c4-draw.io/c4.js"?

McFoggy commented 4 years ago

using 13.3.0 leads in the same CSP issue.

chathuranga commented 4 years ago

Any idea of getting the same "Content Security Policy" issue fixed in the Draw.io plugin version?

@chathuranga Do you have this issue with the http://draw.io Variant and adding C4 Plugin via "https://tobiashochguertel.github.io/c4-draw.io/c4.js"?

It seems to work now(actually it have worked with locally cached JS, failed to work after a refresh). I'm accessing the docs via https://app.diagrams.net/# But I have a doubt, previously we used to have the color coding for the "Software System" vs "External Software System", etc Did we miss that during the updates? Note: even though the functionality is working. I'm still seeing the "Refused to load the script 'https://tobiashochguertel.github.io/c4-draw.io/c4.js' because it violates the following Content Security Policy directive" in browser console.

tobiashochguertel commented 4 years ago

@chathuranga this needs more investigation from me. I have trouble to understand drawio's source code and the documentation, I also searched a different diagram framework which I can use to create this / or a c4-(draw.io) plugin.

So at the moment you are unable able to use c4-draw.io plugin with http://draw.io/, but with desktop version 13.2.2 it works? and with desktop version 13.3.0 it doesn't work?

😩

danilobreda commented 4 years ago

After installing and using 13.3.8 Desktop version, worked... the online version does not

ErikHen commented 4 years ago

It haven't worked for me in the online version for some time, but today I got it working by removing the plugin completely, refreshing the tab and adding the plugin again!

I can also confirm that it is working in draw.io desktop app (13.6.2).

Thanks for all the work that have been put into creating this plugin! Every person who contributes to an open source project of this kind is a true hero 😊.

Jikodis commented 1 year ago

A couple years later I know. I am able to use the plugin within https://app.diagrams.net/ , but I am unable to use it in the desktop version of diagrams.net Version 20.3.0

There is a permanent message in this version of the desktop app that says "External Plugins: External plugins disabled."

I think they are taking a different approach to loading external plugins, meaning they don't seem to load them at all.

I think an alternative would be to communicate with diagrams.net directly and try to get this plugin included by default in their list of built-in plugins.