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

Regression with names #75

Open
Seraf opened this issue Jun 10, 2014 · 5 comments
Open

Regression with names #75

Seraf opened this issue Jun 10, 2014 · 5 comments
Labels

Comments

@Seraf
Copy link

Seraf commented Jun 10, 2014

I'm using the 1.3 version, and I can't display endpoint with / in their names :

Here is my extra_actions :

 extra_actions = [
            {
                'name': 'engine/reload',
                'http_method': 'GET',
                'fields': {}
            },
            {
                'name': 'scheduler/reload',
                'http_method': 'GET',
                'fields': {}
            },
            {
                'name': 'intents',
                'http_method': 'GET',
                'fields': {}
            },
]

The intents endpoint will display with no problems, but the two first will crash with a javascript error :
Uncaught Error: Syntax error, unrecognized expression: /google_post_4_content jquery-1.8.0.min.js:2
Z.error jquery-1.8.0.min.js:2
bg jquery-1.8.0.min.js:2
Z.compile jquery-1.8.0.min.js:2
bm jquery-1.8.0.min.js:2
h.querySelectorAll.bm jquery-1.8.0.min.js:2
Z jquery-1.8.0.min.js:2
p.fn.extend.find jquery-1.8.0.min.js:2
p.fn.p.init jquery-1.8.0.min.js:2
p jquery-1.8.0.min.js:2
OperationView.toggleOperationContent swagger-ui.js:1692
p.event.dispatch jquery-1.8.0.min.js:2
g.handle.h

@johnraz
Copy link
Contributor

johnraz commented Jun 10, 2014

From the distance it looks more like a swagger-ui error :(
The traceback seems cut-off by the way ;)

@Seraf
Copy link
Author

Seraf commented Jun 10, 2014

It's the outpout of my chrome browser.
You're right, I tried to rollback to swagger 1.2 and have the same bug.
It was working last year, maybe a regression on the swagger side. I will
try it again when swagger 2.0 will be merged into django-tastypie-swagger :D

Thanks

On Tue, Jun 10, 2014 at 9:54 AM, Jonathan Liuti [email protected]
wrote:

From the distance it looks more like a swagger-ui error :(
The traceback seems cut-off by the way ;)


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

Julien Syx
Ingénieur Systèmes & Réseaux

@Seraf Seraf closed this as completed Jun 10, 2014
@johnraz
Copy link
Contributor

johnraz commented Jun 10, 2014

Ack, I'd rather keep the issue open so we don't lose track of it.

@johnraz johnraz reopened this Jun 10, 2014
@johnraz johnraz added the bug label Jun 10, 2014
@skakri
Copy link

skakri commented Jul 7, 2014

Hey, I've just set up django-tastypie-swagger and I'm having the same issue. @Seraf maybe you have found a workaround?

@Seraf
Copy link
Author

Seraf commented Jul 7, 2014

Hi @skakri , I didn't found workaround except named differently my endpoints.
But @johnraz is right, it's an issue in the javascript

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

No branches or pull requests

3 participants