drf-tracking icon indicating copy to clipboard operation
drf-tracking copied to clipboard
verified publisher icon aschn

Fix missing query_params on Put/Post/Patch requests. Update tox tests to include django 2.2.6 and drf 3.10.3 versions

Open lingster opened this issue 6 years ago • 5 comments

Hi, I'd like to submit a PR to update this package. Also would you like some help to push a new version to pypi?

lingster avatar Oct 22 '19 20:10 lingster

@lingster I can do my best. I haven't pushed a release in a while but can look into it.

avelis avatar Oct 22 '19 20:10 avelis

@avelis thanks for replying, created the following PR: #143. Unit tests are passing. Pushing a release should also resolve #134

lingster avatar Oct 23 '19 08:10 lingster

@avelis I've also included in the PR fix and tests for an issue where the query params are not stored for PUT/POST/PATCH requests. could you kindly include this for the release?

lingster avatar Oct 23 '19 14:10 lingster

@lingster Can you look into why there is one failing test. Looks to be only on one configuration. If it's benign I can merge it in. https://travis-ci.org/aschn/drf-tracking/jobs/601860781?utm_medium=notification&utm_source=github_status

avelis avatar Oct 23 '19 15:10 avelis

@avelis that should be fixed. I don't have multiple python versions on my system so i am relying on the CI to confirm everything is working.

lingster avatar Oct 23 '19 15:10 lingster