Joe Stump

Results 71 comments of Joe Stump

Hey Vince, If you post some code in a gist, I might be able to comment. The application doesn't integration with Django's CSRF or is_ajax() stuff, but it definitely should....

Correct, `ManyToManyField`'s are not supported currently. This is another example of why we should probably consider a pure JSON payload over HTTP POST as well.

I hadn't even heard of `select_related` before you mentioned this, so I have no idea. What I do know is the `expand` option on `encode_record` will do what you're looking...

Probably not. If you checked for both `ForeignKey` and `OneToOne` in this line of code, it might work: https://github.com/joestump/django-ajax/blob/master/ajax/endpoints.py#L39

I was planning on building Django's CSRF tokens into the entire system so that we can use those tokens automatically within the framework. I haven't gotten around to it yet...

You can, however, check for the CSRF stuff by overriding `ModelEndpoint.authenticate()` and checking the `request` that's passed to that for CSRF stuff.

Yeah, I should probably do that. :smile:

Can you submit that fix as a separate pull request so I can get it merged in?

@tracypholmes this is a feature that's only available in GitHub Enterprise. I don't believe the main GitHub website supports this API endpoint. I ran the tests against our GHE install...

Really could have used this today for the Heroku data provider.