Skip to content
This repository was archived by the owner on Mar 10, 2025. It is now read-only.

Implement write for NestedHyperlinkedRelatedField#2

Open
ssaavedra wants to merge 3 commits into
raphendyr:reimplement_nested_routesfrom
ssaavedra:master
Open

Implement write for NestedHyperlinkedRelatedField#2
ssaavedra wants to merge 3 commits into
raphendyr:reimplement_nested_routesfrom
ssaavedra:master

Conversation

@ssaavedra
Copy link
Copy Markdown

It was needed to set NestedHyperlinkedRelatedFields as read_only just
because they were missing the proper get_object.

An implementation of get_object for view_kwargs' links is provided.

decentral1se and others added 2 commits September 22, 2016 12:11
It was needed to set NestedHyperlinkedRelatedFields as read_only just
because they were missing the proper get_object.

An implementation of get_object for view_kwargs' links is provided.
@ssaavedra
Copy link
Copy Markdown
Author

You can just rebase my branch instead of merging through here; I just implemented the write semantics for NestedHyperlinkedRelatedFields, which were missing and quite a big deal for not much implementation effort 😄

We cannot enable use_pk_only_optimization arbitrarily just using the
lookup_field, because our lookup_map can be used in get_url for
recrafting the URL from more paramters, so we cannot deal with
PKOnlyObjects.
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.

2 participants