improve china account template #15021

Open
wants to merge 3 commits into
from

Projects

None yet

2 participants

@jeffery9
Contributor

Description of the issue/feature this PR addresses:

Current behavior before PR:

Desired behavior after PR is merged:

try to fix #14999

I confirm I have signed the CLA and read the PR guidelines at www.odoo.com/submit-pr

jeffery9 added some commits Jan 11, 2017
@jeffery9 jeffery9 improve china account template
8d24457
@jeffery9 jeffery9 improve account code 367a4e0
+ <!--
+ Copyright (C) 2008-2008 凯源吕鑫 lvxin@gmail.com <basic chart data>
+ Copyright (C) 2008-now 开阖软件 jeff@osbzr.com < PM and LTS >
+ Copyright (C) 2017-now jeffery9@gmail.com
@mart-e
mart-e Jan 11, 2017 Member

please avoid putting copyright notice in every file, you can put it in the description of the module if needed

@jeffery9
jeffery9 Jan 12, 2017 Contributor

ok,

addons/l10n_cn_standard/__manifest__.py
@@ -14,17 +14,21 @@
'maintainer': 'jeff@osbzr.com',
'website': 'http://openerp-china.org',
'description': """
+包含企业会计准则以下数据
@mart-e
mart-e Jan 11, 2017 Member

Could you also add a description in English (can have both)? This way, non-speakers can understand what this module is about.

@jeffery9
jeffery9 Jan 12, 2017 Contributor

nice advice, I should rewrite those message

@mart-e
Member
mart-e commented Jan 11, 2017

Hello,

Thank you for the big changes. However, I am a bit afraid this is too risky for a stable version.
Could you detail what you have changed exactly? Is it only moving from one file to several ones? Is there some records that have been removed?

@jeffery9 jeffery9 arrange and move copyright into manifest and __init__, add english de…
…scription, increase version number.
f123830
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment