Julian Early

Results 17 comments of Julian Early

@bellini666 @Luferov @taekyun-quriously @allangz There is a link to the [graphene slack channel](https://join.slack.com/t/graphenetools/shared_invite/enQtOTE2MDQ1NTg4MDM1LTA4Nzk0MGU0NGEwNzUxZGNjNDQ4ZjAwNDJjMjY0OGE1ZDgxZTg4YjM2ZTc4MjE2ZTAzZjE2ZThhZTQzZTkyMmM) in the `README`, I have also added it here. There is a `django` channel in there @zbyte64...

@jkimbo Is there any update on adding the `Auth` subclass to `DjangoObjectType`? That idea seemed really promising as a built in way to handle this issue

I think the issue is that client updates only happen if there is a token updater set (see https://github.com/requests/requests-oauthlib/blob/master/requests_oauthlib/oauth2_session.py#L537). If we move the `self._client.add_token` call outside of the `if` check,...

I have address both of your comments @bellini666 @la4de I am ready going to go write some tests, but I am realizing that with the change that @la4de suggested, none...

@bellini666 Thinking about your comment, would it be better to replace instances of `Point` with `Union[Tuple[float, float], Tuple[float, float, float]]`? Just do away with using NewType all together

> > I am ready going to go write some tests, but I am realizing that with the change that @la4de suggested, none of these will build without adding `gdal`...

@la4de Can I ask you to run tests on this? I want to check that Spatialite will work properly in CI since it was a pain for me to get...

@la4de Found a way for it to work without needed Spatialite (since we don't have to write the GeosFields to the DB). It should be ready to test and review

I would love to have an option like this, however I don't think that this is ideal. My issue with this format is that you have to enter the same...

@bellini666 Is there anything I can do to help porting that stuff over to the core package? All of those features are much needed