Community mailing list archives

Re: New module for Report RTL printing

Eric Caudal
- 07/18/2015 01:48:02
Hi Mohamed,
Great contribution here!

If you want to have your modules hosted and maintained within the OCA, please send a mail to, asking for the creation of the repository l10n_saudi-arabia.
If your contribution is generic enough, we might think about adding l10n_arabic or l10n_rtl (please give your input/suggestion here as we might not be familiar with the constraints linked to your localization).
After that you would be designated as project leader (given your level of current contribution, sounds logical) and you will be able to clone the repo, push your module in it and commit.
Next step will be a code review, necessary to have the final commit.
Dont forget the basic rules of the OCA about coding: unit tests, PEP8, standard description in README and in general Odoo and python standard programming patterns ad usage.
Additionally, having a simple user manual in the description is welcome.
Looking forward to reading you

On Sat, Jul 18, 2015 at 1:26 AM Mohamed M. Hagag <> wrote:

for sure and with pleasure, but how can I do a PR from my repo to OCA repo ?

On 07/17/2015 11:24 AM, Jordi Ballester Alomar wrote:
<blockquote cite="" type="cite">

Dear Mohamed,

This is great work that you're publishing. I suggest that in order to maximize the value of your work to the Community you submit a Pull Request with the proposed modules to the Odoo Community Association (OCA) repositories.


El dia 16/07/2015 20:22, "Mohamed M. Hagag" <> va escriure:


If you need to print reports in RTL for Arabic and other RTL languages .

This one is very simple and only modifies the report model plus 2 views ( one for web_report.layout ) , also it should fix some issues in Barsi's report_rtl .

Also this is a standalone - doesn't depend on web_rtl nor website_rtl modules.

available here:


If you installed website module, you will need report_rtl_website to get RTL reports working.


Post to:


Post to: