-
Notifications
You must be signed in to change notification settings - Fork 5
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
Import URL does not work #2
Comments
As I'm not much of a JavaScript developer, I don't have an alternative as of yet to the |
I'm going to leave this issue open and perhaps some refactoring can be done to the |
I ran some debugging on the docker extension (on the older version 0.0.4) docker extension dev debug noelm/swagger-editor-extension:0.0.4 This displays dev tools and when I try to Import URL I get the following message:
So looks like react is preventing the use of prompt and another method will have to be used to implement the import URL functionality when Swagger Editor is used within a docker extension (as mentioned in my previous comment could probably look implementing this is a same way as |
In version |
I was thinking about contributing to the swagger-editor code to fix this up and found it was already done in the |
Hi @aslafy-z , Thanks for taking an interest in this project. While I'm aware of the |
The Import URL menu option does not work with the Swagger Editor Docker extension.
It appears that the call to JavaScript
prompt
method does not work within a Docker extension setting.This is the line of code that I think is causing the issue:
L95 in topbar.jsx
The text was updated successfully, but these errors were encountered: