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

Allow creation of partners from *Create and Edit*. #198

Merged
merged 3 commits into from Nov 12, 2015
Merged

Allow creation of partners from *Create and Edit*. #198

merged 3 commits into from Nov 12, 2015

Conversation

yajo
Copy link
Member

@yajo yajo commented Nov 2, 2015

This fixes #78 and adds new tests for it.

This fixes #78 and adds new tests for it.

def setUp(self):
return super(CompanyCase, self).setUp()
self.values.update(lastname=self.values["name"], firstname=False)

Choose a reason for hiding this comment

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

sentence after return

@moylop260
Copy link

after fix travis builds 👍

partner_firstname/tests/base.py:74:80: E501 line too long (81 > 79 characters)
partner_firstname/tests/test_create.py:66: [W0101(unreachable), CompanyCase.setUp] Unreachable code

@yajo
Copy link
Member Author

yajo commented Nov 3, 2015

Thanks @moylop260, it should be ready now.

@rafaelbn
Copy link
Member

Tested and fixed 👍

@pedrobaeza
Copy link
Member

👍

pedrobaeza added a commit that referenced this pull request Nov 12, 2015
Allow creation of partners from *Create and Edit*.
@pedrobaeza pedrobaeza merged commit 4adb55c into OCA:8.0 Nov 12, 2015
@alexis-via
Copy link
Contributor

@yajo Thank you very much for this bug fix ! You are THE man !

@yajo yajo deleted the fix-78 branch November 18, 2015 11:02
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.

None yet

5 participants