Skip to content
This repository was archived by the owner on Oct 1, 2022. It is now read-only.

Added queryparams to dump_data view and added test for it. #44

Merged

Conversation

pconerly
Copy link
Contributor

@semente @jaap3

For #30

This is only the multiple app_label endpoint for the /admin/smuggler/ page. I think it makes sense to just overload the /admin/dump/ aka dump_data view to have an optional app_label feature.

@jaap3 jaap3 merged commit 3fa4d75 into semente:master Aug 19, 2014
@jaap3
Copy link
Collaborator

jaap3 commented Aug 19, 2014

Merged with some cleanup and a python 3 fix to the tests, thanks!

@pconerly
Copy link
Contributor Author

Thanks! This is the main feature I needed-- do you have the ability to deploy to pypi? Having smuggler 0.4.2 come out this week would be great.

What features or tickets do you want to finish before publishing 0.4.2?

@semente
Copy link
Owner

semente commented Aug 19, 2014

I will try release this today or tomorrow. Would be great if it could be tested with Django 1.7 too.

@jaap3
Copy link
Collaborator

jaap3 commented Aug 19, 2014

I'm away from the office till thursday so I won't be able to do anything until then. Testing Django 1.7 is just a matter of adding it to the tox config. Hopefully everything keeps working ;). I've already updated setup.py and the readme in anticipation of a new release (0.5).

@pconerly
Copy link
Contributor Author

Thursday is fine. Sweet. I'm running django1.7 tox tests as we speak, PR for you momentarily.

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

Successfully merging this pull request may close these issues.

3 participants