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

fist commit module report_stock_spreadsheet #20

Closed
wants to merge 30 commits into from

Conversation

miguelchuga
Copy link

Hi

this is a report stock to spreadsheet

@oca-clabot
Copy link

Hey @miguelchuga, thank you for your Pull Request.

It looks like some users haven't signed our Contributor License Agreement, yet.
You can read and sign our full Contributor License Agreement here: http://odoo-community.org/page/website.cla
Here is a list of the users:

Appreciation of efforts,
OCA CLAbot

@moylop260
Copy link

@@ -0,0 +1,661 @@
GNU AFFERO GENERAL PUBLIC LICENSE

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

@miguelchuga
Can you see my last comment here?

Copy link
Author

Choose a reason for hiding this comment

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

Hi, @moylop260 last resolved changes

Choose a reason for hiding this comment

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

@miguelchuga
You have LICENSE file, yet

@@ -0,0 +1,33 @@
sudo: false

Choose a reason for hiding this comment

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

The file .travis.yml should be in root path of project.

@miguelchuga
Copy link
Author

Ready.

@oca-clabot
Copy link

Hey @miguelchuga,
We acknowledge that the following users have signed our Contributor License Agreement:

Appreciation of efforts,
OCA CLAbot

@miguelchuga
Copy link
Author

this is a new version

@miguelchuga
Copy link
Author

ready

@miguelchuga
Copy link
Author

Travis

@@ -0,0 +1,32 @@
sudo: true

Choose a reason for hiding this comment

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

@miguelchuga
You need modify the .travis.yml from root path.
This .travis.yml there is in your module then is unused

@moylop260
Copy link

@miguelchuga
Can you see the result of runbot?
It is red still

@moylop260
Copy link

@miguelchuga
Can you add unit-test for this module to check that el report is correctly printed?

Bugs are tracked on `GitHub Issues <https://github.com/OCA/stock-logistics-reporting/issues>`_.
In case of trouble, please check there if your issue has already been reported.
If you spotted it first, help us smashing it by providing a detailed and welcomed feedback
`here <https://github.com/OCA/stock-logistics-reporting/issues/new?body=module:%20stock_picking_partner_lang%0Aversion:%208.0.1.0.0%0A%0A**Steps%20to%20reproduce**%0A-%20...%0A%0A**Current%20behavior**%0A%0A**Expected%20behavior**>`_.

Choose a reason for hiding this comment

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

Use your module name report_stock_spreadsheet instead of `stock_picking_partner_lang``

@moylop260
Copy link

You branch have 30 new commits.
Rebase, please
git rebase -i --autosquash origin/9.0
And replace pick by fixup in all commits except in first one.

@moylop260
Copy link

Edit the name of this PR to explain a resume of that is doing.

@moylop260
Copy link

@pedrobaeza
Please, Could you install in runbot the package: pip install openpyxl?

.. image:: https://img.shields.io/badge/licence-AGPL--3-blue.svg
:target: http://www.gnu.org/licenses/agpl-3.0-standalone.html
:alt: License: AGPL-3

Choose a reason for hiding this comment

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

Add try-on-runbot section like as original template

@rousseldenis
Copy link
Sponsor Contributor

I will close this as it's very old. Feel free to reopen it if needed

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

4 participants