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

[11.0] [MIG] contract sale #145

Merged
merged 4 commits into from
Aug 23, 2018
Merged

Conversation

levkar
Copy link
Contributor

@levkar levkar commented Feb 16, 2018

No description provided.

@levkar levkar changed the title 11.0 mig contract sale [11.0] [MIG] contrat sale Feb 16, 2018
@levkar levkar changed the title [11.0] [MIG] contrat sale [11.0] [MIG] contract sale Feb 16, 2018
@pedrobaeza pedrobaeza added this to the 11.0 milestone Feb 16, 2018
@pedrobaeza pedrobaeza mentioned this pull request Feb 16, 2018
8 tasks
@@ -0,0 +1,55 @@
.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
Copy link
Member

Choose a reason for hiding this comment

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

@levkar Please update readme with png images, runbot Odoo version..

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I migrated according to the guide and there's no such thing for png images.

https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-11.0

Maybe we should update the migration guide so that this is done automatically.

Copy link
Member

Choose a reason for hiding this comment

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

There's already a point about that:

* Update README.rst from https://raw.githubusercontent.com/OCA/maintainer-tools/master/template/module/README.rst if not updated to the latest template.

],
'license': 'AGPL-3',
'installable': True,
'auto_install': True,
Copy link
Member

Choose a reason for hiding this comment

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

@levkar Why the autoinstall...

Copy link
Contributor Author

@levkar levkar Mar 30, 2018

Choose a reason for hiding this comment

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

I don't know. It was like that in 10.0.

Copy link
Member

Choose a reason for hiding this comment

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

It should be that way, as this glue module is needed if you have sale installed.

@pedrobaeza
Copy link
Member

Please squash adjacent "OCA Transbot..." commits using https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests

@pedrobaeza
Copy link
Member

Any news about this?

@levkar
Copy link
Contributor Author

levkar commented May 19, 2018

Squash done.


.. image:: https://odoo-community.org/website/image/ir.attachment/5784_f2813bd/datas
:alt: Try me on Runbot
:target: https://runbot.odoo-community.org/runbot/110/10.0
Copy link
Member

Choose a reason for hiding this comment

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

11.0

'Odoo Community Association (OCA)',
'website': 'https://github.com/OCA/contract',
'depends': [
'sale_management',
Copy link
Member

Choose a reason for hiding this comment

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

Instead of 'sale_management', the dependency with 'sale' is enough. Could you put this to 'sale'?

* README by fragments
* Adjusted dependency
* Changed root menu XML-ID
@pedrobaeza pedrobaeza merged commit 63fc8b8 into OCA:11.0 Aug 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants