-
Notifications
You must be signed in to change notification settings - Fork 2
Public preview #6
base: master
Are you sure you want to change the base?
Conversation
action: "load", | ||
xml: data | ||
}), "*"); | ||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Add error handling
js/public.js
Outdated
$(document).ready(function(){ | ||
if ($('#isPublic').val()) { | ||
// TODO this needs to be imported | ||
var src = 'https://www.draw.io?embed=1&chrome=0&spin=1&stealth=1&proto=json'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Pass in this url from somewhere??
lib/Application.php
Outdated
|
||
$manager = \OC::$server->getContentSecurityPolicyManager(); | ||
$policy = new ContentSecurityPolicy(); | ||
$policy->addAllowedFrameDomain('https://www.draw.io'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Unify this url config
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Remove csp from page controller as it is handled here now
drawio.tar.gz |
Tested ;) |
Does this PR cause #8 or is that something different? |
Greetings, came to this issue because I tried to share a diagram as a public link without password, and it seems to not work yet. I'm using v0.9.4 under NextCloud v17. Congratulations for all the hard work. |
@digitall-it Well - this app is made for ownCloud :) |
TODO: