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

add OCA author #14

Merged
merged 3 commits into from Sep 8, 2015
Merged

add OCA author #14

merged 3 commits into from Sep 8, 2015

Conversation

ntodorova
Copy link
Member

Add OCA as author.

@yvaucher
Copy link
Member

yvaucher commented Sep 7, 2015

@ntodorova Thanks, however you need to put it in the same string, apps.odoo.com doesn't support list but split on coma.

@OSguard
Copy link

OSguard commented Sep 7, 2015

a other way is:

    'Odoo Community Association (OCA),'
    'initOS GmbH & Co. KG',

that looks clear and works well

@OSguard
Copy link

OSguard commented Sep 7, 2015

@yvaucher now the space after the comma is not needed or?

@hbrunn
Copy link
Member

hbrunn commented Sep 7, 2015

@OSguard you need the comma, but no space

@OSguard
Copy link

OSguard commented Sep 7, 2015

sorry

    'Odoo Community Association (OCA),'
    'initOS GmbH & Co. KG',

is what i was looking for

@yvaucher
Copy link
Member

yvaucher commented Sep 7, 2015

👍 either for current state or the one proposed by @OSguard

@OSguard
Copy link

OSguard commented Sep 7, 2015

👍

1 similar comment
@dreispt
Copy link
Sponsor Member

dreispt commented Sep 7, 2015

👍

yvaucher added a commit that referenced this pull request Sep 8, 2015
@yvaucher yvaucher merged commit 8f15caf into OCA:7.0 Sep 8, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants