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

Update __init__.py #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

625781186
Copy link

admin list column name show verbose_name

admin list column name show verbose_name
@PetrDlouhy
Copy link
Owner

I have reworked the testing on this project to use GitHub actions. Could you please rebase this code to the current master to run the tests?

I am not sure, it in order to achieve this, we need such extensive monkey-patching. There already is getter_for_related_field which allows this to be done for individual fields much easier way.
Could you please describe why this is not possible to achieve with less code copied from Django?

@625781186
Copy link
Author

625781186 commented Aug 9, 2023

I have reworked the testing on this project to use GitHub actions. Could you please rebase this code to the current master to run the tests?

I am not sure, it in order to achieve this, we need such extensive monkey-patching. There already is getter_for_related_field which allows this to be done for individual fields much easier way. Could you please describe why this is not possible to achieve with less code copied from Django?

My Django Version : 2.1.5

Actually I'm not very proficient in django,
This is just a temporary solution I researched yesterday,
you can implement it in a better way .

This code is django source code ,
link is https://github.com/django/django/blob/80a5667c50d12b27a031a9ce0408a5b81bc4e684/django/contrib/admin/utils.py#L341

This is code call stack frame:
#12 (comment)

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