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

[9.0] partner_firstname - Fix user form with firstname and lastname asking for a mandatory name #249

Merged
merged 1 commit into from
Apr 27, 2016

Conversation

yvaucher
Copy link
Member

No description provided.

@yvaucher yvaucher changed the title Fix user form with firstname and lastname asking for a mandatory name [9.0] partner_firstname - Fix user form with firstname and lastname asking for a mandatory name Feb 18, 2016
@yajo
Copy link
Member

yajo commented Feb 19, 2016

It would be nice if you can add a test for this.

@@ -28,3 +28,10 @@ def default_get(self, fields_list):
result[field] = inverted.get(field)

return result

@api.one
Copy link
Contributor

Choose a reason for hiding this comment

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

deprecated

@coveralls
Copy link

coveralls commented Apr 26, 2016

Coverage Status

Coverage decreased (-10.7%) to 86.667% when pulling 1a4ed83 on yvaucher:fix-firstname-user-form into fde972c on OCA:9.0.

@pedrobaeza
Copy link
Member

Please check pylint error

@yvaucher yvaucher force-pushed the fix-firstname-user-form branch from 1a4ed83 to 5744cce Compare April 26, 2016 16:58
@yvaucher yvaucher force-pushed the fix-firstname-user-form branch from 5744cce to 17d8619 Compare April 26, 2016 17:01
@yvaucher
Copy link
Member Author

@pedrobaeza fixed (don't over do, travis is already there to say so 😄 )

@yajo I just added a test on the onchange of user

@coveralls
Copy link

coveralls commented Apr 26, 2016

Coverage Status

Coverage increased (+0.02%) to 97.436% when pulling 17d8619 on yvaucher:fix-firstname-user-form into fde972c on OCA:9.0.

@pedrobaeza
Copy link
Member

👍

@yvaucher, it was just in case you were not aware of it.

@leemannd
Copy link
Contributor

👍

@pedrobaeza pedrobaeza merged commit 8bb6e7c into OCA:9.0 Apr 27, 2016
@yvaucher yvaucher deleted the fix-firstname-user-form branch April 28, 2016 07:13
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.

5 participants