Sepa xml file about l10n-italy HOT 6 CLOSED

I found "codice_sia" in l10n_it_ricevute_bancarie module.
Is it possible declare this field in some base module, because it should be use by:

• Sepa SCP (aka bonifico bancario SEPA)
• Sepa SDD (aka addebito su conto)
• RiBA (this feature still remain unchanged)

Thanks

from l10n-italy.

It makes sense.
Porting RiBa to 8.0 is still work in progress: #115
So, we can remove that field from riba module, create l10n_it_sia and make l10n_it_ricevute_bancarie depend on l10n_it_sia.
Could you provide a PR for that?

from l10n-italy.

Yes, I could but I have no experience with right flow.
I think I should:

1. Clone project (both branch V7 and V8)
2. Update source with PEP8 (if needed)
3. Make update
4. Issue a PR

Is it right flow?

from l10n-italy.

See Basic Branching and Merging guide.
In this case, you can start branching 8.0 branch in order to add l10n_it_sia module (see the OCA Guidelines) and make a PR (see Using pull requests) to merge your branch into 8.0.
Then, you can post a comment on #115, so that we can decide how to proceed

from l10n-italy.

Thanks.
I will read that documents and then I wil try to operate.
May ask you about, if I will meet some trouble?

from l10n-italy.

Sure, please consider joining us on IRC freenode channels #oca and #odoo-it (http://www.odoo-italia.org/index.php/community-la-comunita-openerp/chat)

from l10n-italy.