Skip to content

Extra attributes problem, fixes #202#215

Merged
benkonrath merged 1 commit intodjango:masterfrom
vladimirnani:extra_fields_problem
Apr 3, 2016
Merged

Extra attributes problem, fixes #202#215
benkonrath merged 1 commit intodjango:masterfrom
vladimirnani:extra_fields_problem

Conversation

@vladimirnani
Copy link
Copy Markdown
Contributor

Problem was that attributes were not passed to formfield validators, only to field validators.

from localflavor.generic.models import IBANField


class Partner(models.Model):
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you rename this to something like UseNordeaExtentions to match IncludedCountriesModel? Same applies to the form.

@benkonrath
Copy link
Copy Markdown
Member

This looks good. Can you rename the classes that I indicated and then rebase / squash your commits into one? Thanks.

@vladimirnani vladimirnani force-pushed the extra_fields_problem branch 4 times, most recently from 9d9bafc to 77b3e25 Compare April 3, 2016 08:50
@vladimirnani vladimirnani force-pushed the extra_fields_problem branch from 77b3e25 to b325234 Compare April 3, 2016 09:00
@benkonrath benkonrath merged commit b325234 into django:master Apr 3, 2016
@benkonrath
Copy link
Copy Markdown
Member

@vladimirnani Thanks!

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