Add Apps store link if OCA

[elicoidal]

cc @OCA/board

[pedrobaeza]

For reducing the diff in all READMEs, switch the order of the sentences. You have to check tests also (there are tests for README generation).

[sbidoul]

Changes to the readme generator must be considered very carefully, because the nightly regeneration job will trigger many commits and republish all modules to pypi.

[elicoidal]

no clear why travis is failing

[elicoidal]

@pedrobaeza I understand the diff problem but I prefer this order which makes more sense to me.
@sbidoul Fully agree and I am open to suggestion: this is a follow up about some discussion with the board about increasing the SEO of the shop.
Maybe we should queue the modifications and wait for another significant one.

[pedrobaeza]

@elicoidal they are unconnected things, so the order is not so important, so for me, they should go the other way for decreasing the diff.

[elicoidal]

@pedrobaeza the logic is "Here is the work we do hence if you want to contribute ..."
The other way around is "Contribute. Here is the work we do"

This is why I propose this order

[pedrobaeza]

Well, I still think the other way is correct also, so I prefer to not switch it.

[elicoidal]

I am not convinced but have no time to fight this

[pedrobaeza]

Please rebase to get a green Travis

[elicoidal]

@pedrobaeza I dont know how to interpret the travis and fix the PR. Can you help me?

[pedrobaeza]

You have to change tests in https://github.com/OCA/maintainer-tools/blob/master/tests/test_gen_addon_readme.py

[elicoidal]

You have to change test

@pedrobaeza Sorry I do not know how to do that... Help is welcome to finish this.

[pedrobaeza]

@sbidoul does this makes sense still?

[sbidoul]

Not sure. Our app store is not in very good shape so I'd hesitate to do this at the moment.

Also, I'm wondering if we should not change the bot to not do README.rst generation in the nightly batch, but only do it in ocabot merge or even pre-commit. Not sure.

[pedrobaeza]

What do you mean by "Our app store is not in very good shape"? I use as my first OCA modules info source with no problems.

[sbidoul]

Well I may overreact but I remember several people reporting problems and there used to be some synchronization issues to keep it up-to-date. Maybe these have been fixed by now.

[pedrobaeza]

I think they were all addressed, but anyways, if any persists, we should fix it (and you can consider that as part of the warranty of the migration), but don't dismiss the "pipelining" to it for avoiding the rest of the issues with Odoo Apps Store.