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

Outdated documentation #6558

Closed
rpkilby opened this issue Apr 2, 2019 · 2 comments
Closed

Outdated documentation #6558

rpkilby opened this issue Apr 2, 2019 · 2 comments

Comments

@rpkilby
Copy link
Member

rpkilby commented Apr 2, 2019

See #5886 (comment)

Just opening this to make sure we don't lose track of it.

The docs recommend

"If you want the date field to be visible, but not editable by the user, then set read_only=True and additionally set a default=... argument."

It then says:

"The field will not be writable to the user, but the default value will still be passed through to the validated_data."

This is not true. It will not be passed through to the validated_data.

@phankiewicz
Copy link
Contributor

Seems like @rpkilby already made this change in #6847. Let's close this ticket then.

@rpkilby
Copy link
Member Author

rpkilby commented Oct 3, 2019

Thanks

@rpkilby rpkilby closed this as completed Oct 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants