Skip to content

Load YAML from URLs with query param #97

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

Closed
mohsen1 opened this issue Aug 26, 2014 · 6 comments
Closed

Load YAML from URLs with query param #97

mohsen1 opened this issue Aug 26, 2014 · 6 comments

Comments

@mohsen1
Copy link
Contributor

mohsen1 commented Aug 26, 2014

theeditor.com?url="theurl" should open the editor for that url

@mohsen1 mohsen1 changed the title Load URLs with query param Load YAML from URLs with query param Aug 27, 2014
@fehguy
Copy link
Contributor

fehguy commented Sep 5, 2014

this would be great for sharing links to swagger specs

http://editor.swagger.wordnik.com/?spec=http://generator.wordnik.com/online/api/swagger.yaml

@mohsen1
Copy link
Contributor Author

mohsen1 commented Sep 5, 2014

Yes, that's the plan

@earth2marsh
Copy link
Contributor

Suggest we call the qparam something other than url… how about import, since it aligns better with the menu action of Import by URL…. Also consider stripping the param after loading, so that a reload doesn't collide with the locally saved state.

(also, I want this badly)

@mohsen1 mohsen1 closed this as completed in c1ac67d Sep 5, 2014
@mohsen1
Copy link
Contributor Author

mohsen1 commented Sep 5, 2014

@earth2marsh @fehguy

This feature has been implemented. Please note that you will need a # before ? because it's an Angular app. For example this url will work:

http://editor.swagger.wordnik.com/#?import=http://generator.wordnik.com/online/api/swagger.yaml

Edit: updating the url to use import instead

@mohsen1
Copy link
Contributor Author

mohsen1 commented Sep 5, 2014

@earth2marsh Now it's striping the query param. With 4678015

@earth2marsh
Copy link
Contributor

… and the README.md has been updated to document it!

On Fri, Sep 5, 2014 at 3:52 PM, Mohsen Azimi [email protected]
wrote:

@earth2marsh https://github.com/earth2marsh Now it's striping the query
param. With 4678015
4678015


Reply to this email directly or view it on GitHub
#97 (comment)
.

tim-lai pushed a commit that referenced this issue Jun 9, 2022
ci: use a curl/api-based Rancher deploy instead of marketplace action
ioggstream added a commit to italia/schema-editor that referenced this issue Nov 14, 2024
* Tests for swagger-api#96.

* Fix: swagger-api#96. Use @context to resolve @type.

* See: swagger-api#63. About nested @context.

* refactory classUri not valid

---------

Co-authored-by: Stefano Baruzzo <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants