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

[7.0] Add module 'partner_address_street_3' #126

Merged
merged 6 commits into from
Jan 18, 2016

Conversation

gurneyalex
Copy link
Member

backport the addon from 8.0

rebased #60

@coveralls
Copy link

Coverage Status

Coverage increased (+0.45%) to 44.46% when pulling b23b18e on gurneyalex:7.0-street3 into ed7b779 on OCA:7.0.

@pedrobaeza
Copy link
Member

Two PEP8 issues:

./partner_address_street3/model/partner.py:42:5: E303 too many blank lines (2)
./partner_address_street3/model/partner.py:60:1: W391 blank line at end of file

A README.rst file is missing.

@gurneyalex
Copy link
Member Author

failure on runbot is caused by unrelated warnings.

Contributors
------------

* Firstname Lastname <email.address@example.org>
Copy link
Contributor

Choose a reason for hiding this comment

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

Who are the contributors?

@antespi
Copy link
Contributor

antespi commented Dec 15, 2015

👍 After minor remarks fixed

{
'name': 'Street3 in addresses',
'version': '0.1',
'author': 'Camptocamp',

Choose a reason for hiding this comment

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

Add OCA as author so it shows up in the odoo app store

##############################################################################
{
'name': 'Street3 in addresses',
'version': '7.0.0.1.0',
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be 7.0.1.0.0?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'm using the same number as the 8.0 version from which this was ported.

Copy link
Member

Choose a reason for hiding this comment

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

Well, OK, usually the first version arriving to OCA has the 1.0.0 version milestone, but not a blocking issue.

@pedrobaeza
Copy link
Member

👍

pedrobaeza added a commit that referenced this pull request Jan 18, 2016
[7.0] Add module 'partner_address_street_3'
@pedrobaeza pedrobaeza merged commit cb37725 into OCA:7.0 Jan 18, 2016
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

7 participants