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

Simplified basic usage example #56

Closed
wants to merge 1 commit into from
Closed

Conversation

melinath
Copy link

ListBulkCreateUpdateDestroyAPIView was being used in the basic example. In most cases, I think people are more likely to be using ViewSets. The BulkModelViewSet is only tangentially mentioned in the README, and the example given looks similar enough to a viewset that it is confusing. People who have a use case that requires a specific API View are probably more likely to be willing to dig, and more likely to notice that BulkModelViewSet is a viewset rather than a view.

tldr; perhaps optimize for people who are just trying to get it to work in the first place?

@melinath
Copy link
Author

This is related to #55

@melinath
Copy link
Author

Although actually the BulkModelViewSet doesn't work out of the box either... seems to need extra routes defined.

@melinath
Copy link
Author

orrrr maybe it just needs the bulk router. :-p

@melinath melinath closed this Oct 25, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant