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

Add a way to use properties on list_display elements #11191

Closed
wants to merge 1 commit into from
Closed

Add a way to use properties on list_display elements #11191

wants to merge 1 commit into from

Conversation

LeeHanYeong
Copy link
Contributor

In Django admin, you can use the property of the Model, but there is no description in the documentation.
I've added descriptions and examples

@LeeHanYeong
Copy link
Contributor Author

I have successfully run the make html command locally, but I do not know why the build fails.
I would appreciate your help.

@felixxm
Copy link
Member

felixxm commented Apr 11, 2019

@LeeHanYeong Using property in the Django admin is described in the paragraph that you're trying to changed. Limitation with @property decorator is also described in this paragraph

... on a property is possible when using the ``property()`` function and not with the
@property decorator.

I don't think that we should add to the documentation workarounds with assigning values to the fget.

@felixxm felixxm closed this Apr 11, 2019
@LeeHanYeong LeeHanYeong deleted the lhy-docs-admin-property branch April 20, 2022 09:55
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.

2 participants