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

Refs #24121 -- Added __repr__() to AdminForm, BlockContext, BlockTranslateNode, and IncludeNode. #14504

Conversation

saeedblanchette
Copy link
Contributor

Hello.
I see that InlineAdminFormSet inherits from AdminForm so I think we don't need to add to it __repr__
Also, I'm not sure where I should put those tests.
Thank you for reviewing

@felixxm felixxm self-assigned this Jun 9, 2021
@felixxm felixxm force-pushed the Refs#24121--Added__repr__()toIncludeNode,BlockContext,AdminForm,InlineAdminFormSet,andBlockTranslateNode branch from 346e0bb to 3d17968 Compare June 10, 2021 10:44
Copy link
Member

@felixxm felixxm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@saeedblanchette Thanks 👍

I moved tests to the correct files and pushed small edits.

@felixxm felixxm changed the title Refs #24121 -- Added __repr__() to IncludeNode, BlockContext, AdminForm, and BlockTranslateNode Refs #24121 -- Added __repr__() to AdminForm, BlockContext, BlockTranslateNode, and IncludeNode. Jun 10, 2021
@felixxm felixxm force-pushed the Refs#24121--Added__repr__()toIncludeNode,BlockContext,AdminForm,InlineAdminFormSet,andBlockTranslateNode branch from 3d17968 to 66ed03e Compare June 10, 2021 10:48
@felixxm felixxm merged commit 66ed03e into django:main Jun 10, 2021
@saeedblanchette saeedblanchette deleted the Refs#24121--Added__repr__()toIncludeNode,BlockContext,AdminForm,InlineAdminFormSet,andBlockTranslateNode branch February 18, 2022 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants