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

French documentation #57

Merged
merged 3 commits into from
May 1, 2015
Merged

French documentation #57

merged 3 commits into from
May 1, 2015

Conversation

ccomb
Copy link
Contributor

@ccomb ccomb commented Apr 16, 2015

Here is an exhaustive french translation of the documentation (even with the API).

We currently have a sphinx build which is generated using this buildout and automatically updated by these builders using this configuration

@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.07% when pulling e2c8b1f on anybox:french-doc into 6686fd3 on OCA:8.0.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage remained the same at 80.07% when pulling e2c8b1f on anybox:french-doc into 6686fd3 on OCA:8.0.

@@ -53,7 +53,7 @@ def do_something_product(session, model_name, a, b):
on_my_event.subscribe(do_something_product,
model_names='product.product')

We can also replace an event::
We can also replace an consumer::
Copy link
Contributor

Choose a reason for hiding this comment

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

:%s/an consumer/a consumer

@lmignon
Copy link
Contributor

lmignon commented Apr 17, 2015

👍 Thank you @ccomb!!!

@ccomb
Copy link
Contributor Author

ccomb commented Apr 17, 2015

I'm adding a short text in the contribute page to explain how to maintain a translation

@guewen
Copy link
Member

guewen commented Apr 21, 2015

Wow, what a massive work!
Thanks @ccomb 😍

$ mkdir buildout && cd buildout
$ wget https://bitbucket.org/anybox/public_buildbot_buildouts/raw/tip/odoo-connector.cfg
$ wget https://bitbucket.org/anybox/public_buildbot_buildouts/raw/tip/bootstrap.py
$ python bootstrap.py && bin/buildout -c odoo-connector.cfg
Copy link
Member

Choose a reason for hiding this comment

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

Following these steps, python bootstrap.py gives: Error: Couldn't open /tmp/tmp/buildout.cfg. We have to download odoo-connector.cfg to buildout.cfg before with wget https://bitbucket.org/anybox/public_buildbot_buildouts/raw/tip/odoo-connector.cfg -O buildout.cfg.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe repeat this step here not to miss it:
Addthis patch https://bitbucket.org/shimizukawa/sphinx-intl/pull-request/9/`_ to sphinx-intl (until merged) to support fuzzy translations`

@guewen
Copy link
Member

guewen commented Apr 21, 2015

@ccomb According to you, what is the better way to publish the french version on odoo-connector.com?
One way I think of is to push it to a /fr subfolder and add a link on the index page saying something like "this documentation is also available in french" directing to /fr.

@gurneyalex
Copy link
Member

just wow!

@guewen
Copy link
Member

guewen commented Apr 21, 2015

Would your builders be able to push the docs on odoo-connector.com (considering it would have access rights) and would you agree to push the builds there? Currently this is a manual process, I git add, git commit and git push the new builds in the repo: https://github.com/camptocamp/openerp-connector-docs (we can move the repo if required) (then the pages are automatically served by GitHub pages).

@ccomb
Copy link
Contributor Author

ccomb commented Apr 21, 2015

@guewen the 'fr/' subfolder is a good idea, since sphinx is still not able to natively host several translations, indeed the index page should provide a list of the available translations pointing to the correct url.
I think our buildbot is capable of uploading the build to a different url, because the upload is just the step 20 of the builder, whose configuration is public and the destination URL is visible in the configuration. Just need to cross-check with @gracinet

@gracinet
Copy link

On 04/21/2015 12:06 PM, Christophe Combelles wrote:

I think our buildbot is capable of uploading the build to a different
url, because the upload is just the step 20
http://buildbot.anybox.fr/builders/connector-postgresql-9.4/builds/7
of the builder, whose configuration is public and the destination URL
is visible
https://bitbucket.org/anybox/public_buildbot_buildouts/src/b1ac741276c21316b20faf1a9372bbcdf605b0c2/MANIFEST.cfg?at=default#cl-16
in the configuration. Just need to cross-check with @gracinet
https://github.com/gracinet

@ccomb, yes, it is, details are here:
http://docs.anybox.fr/anybox.buildbot.openerp/current/apidoc/anybox.buildbot.openerp.subfactories.postbuildout.html?highlight=sphinx#anybox.buildbot.openerp.subfactories.postbuildout.sphinx_doc

@gracinet
Copy link

@guewen previously I had only @comb latest comment on-hand, so I thought it was only a matter of subpath… There's no uploading of any kind currently from our buildmaster to another server (we're cheating, so to speak), but I could add a special step to that effect.

I've never hooked such a bot to GitHub, so if there's a better way than creating a full-blown user account, please tell me

@guewen
Copy link
Member

guewen commented Apr 24, 2015

I've never hooked such a bot to GitHub, so if there's a better way than creating a full-blown user account, please tell me

A user account for pushing the changes you mean? I think that's the way to go.

@guewen
Copy link
Member

guewen commented Apr 29, 2015

@gracinet @ccomb I'm currently investigating the build of the docs from Travis

@guewen
Copy link
Member

guewen commented May 1, 2015

Let get it merged and I will complete #60 to build the french doc as well and include the links.
Thanks again @ccomb

guewen added a commit that referenced this pull request May 1, 2015
@guewen guewen merged commit ad9fa68 into OCA:8.0 May 1, 2015
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.

6 participants